Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WFLY-15978] Enable Elytron security domains for all connectors #15348

Closed
wants to merge 1 commit into from

Conversation

bstansberry
Copy link
Contributor

Signed-off-by: Patrick Reinhart <patrick@reini.net>
@bstansberry bstansberry added the 26.x WildFly 26 label Mar 18, 2022
@bstansberry
Copy link
Contributor Author

@tadamski @reinhapa I ported this to the 26.x branch for possible inclusion in WF 26.1.

@fjuma fjuma added the hold PR should not be merged for some reason. label Apr 1, 2022
@darranl
Copy link
Contributor

darranl commented Apr 6, 2022

I am going to close for now as the upstream fix is being reviewed, we can either reopen or create a new backport if we think the resulting changes could be included in WildFly 26.1.1

@darranl darranl closed this Apr 6, 2022
@reinhapa
Copy link
Contributor

reinhapa commented Apr 6, 2022

@darranl sad, so I will have to patch it in this case...

@tadamski
Copy link
Contributor

tadamski commented Apr 7, 2022

@reinhapa I'm working on alternative PR

@reinhapa
Copy link
Contributor

@tadamski any chance to have this in a later 26.1.1?

@tadamski
Copy link
Contributor

tadamski commented May 4, 2022

hi @reinhapa
I have opened PRs which move Elytron configuration metadata to IronJacamar code:
ironjacamar/ironjacamar#750
#15440
The idea behind those is that you can mark IJ resources as Elytron enabled by adding 'elytron' tag in the configuration. This way such configurations can work with new WildFly/EAP and be verified as Elytron based. OTOH we are avoiding transforming all legacy configs to Elytron under the hood (possibly without the knowledge of the deployment creator).

@reinhapa
Copy link
Contributor

reinhapa commented Jun 1, 2022

I have opened PRs which move Elytron configuration metadata to IronJacamar code:
@tadamski I saw that you're PR ironjacamar/ironjacamar#750 has been closed. Did it not make it into the IronJacamar code base? Does this mean, that my change has been rolled back and no alternative is available for now?

@tadamski
Copy link
Contributor

hi @reinhapa
I have reopend those PR as draft ones:
ironjacamar/ironjacamar#754
#15556

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
26.x WildFly 26 hold PR should not be merged for some reason.
Projects
None yet
5 participants