ELY-2872 - Security roles lost following failover#2247
ELY-2872 - Security roles lost following failover#2247pedro-hos wants to merge 1 commit intowildfly-security:2.6.xfrom
Conversation
|
@pedro-hos Thanks for your PR! Just noticed that the PR is currently referencing a WFLY issue instead of an ELY issue. Please move the WFLY issue to the ELY project and update the PR title and commit to reference the ELY issue instead. Please also submit this PR against the 2.6.x branch instead of the 2.x branch. Thanks! |
pedro-hos
changed the title
@fjuma Thank you for your feedback. I think that this is ok now. I moved from 2.x to 2.6.x and created the new ELY issue |
|
@pedro-hos FYI this PR currently have a merge conflict, but also I think we are ready to look at where a test case can live for this scenario - the changes I made previously to Elytron Web may be possible to be extended to add this kind of scenario. |
skyllarr
left a comment
skyllarr
left a comment
There was a problem hiding this comment.
Just to note here that a PR with a test is here: wildfly-security/elytron-web#285
wildfly-elytron/pom.xml
Outdated
| <groupId>org.wildfly.security</groupId> | ||
| <artifactId>wildfly-elytron</artifactId> | ||
| <version>2.6.3.Final</version> <!-- When updating to next release, clear all exclusions. --> | ||
| <version>2.6.4.Final</version> <!-- When updating to next release, clear all exclusions. --> |
There was a problem hiding this comment.
Hi @pedro-hos , this PR should not change the version here
|
@pedro-hos I needed to rewrite a commit message here so it contains ELY issue number and add a fix on top of this. So I submitted #2294 that supersedes this PR |
|
@skyllarr May we close this one so? |
|
Superseded by #2294 |
1 similar comment
|
Superseded by #2294 |
4 participants
Uh oh!
There was an error while loading. Please reload this page.