[ELYWEB-222] Add a test for single sign on across two apps with logout #245
Conversation
fjuma
force-pushed
the
ELYWEB-222-1.9.x
branch
from
03c7a88 to
1ee6a25
Compare
fjuma
changed the title
| String cacheManagerName = UUID.randomUUID().toString(); | ||
| EmbeddedCacheManager cacheManager = new DefaultCacheManager( | ||
| GlobalConfigurationBuilder.defaultClusteredBuilder() | ||
| .globalJmxStatistics().cacheManagerName(cacheManagerName) |
There was a problem hiding this comment.
FYI may need to double check the PR I sent with the component upgrades, I think I made a small tweak to set up due to Infinispan changes.
There was a problem hiding this comment.
Right, this will need a small tweak when merging to the 4.x branch. The 1.9.x maintenance branch doesn't contain the component upgrade so this passes for me locally.
| } | ||
|
|
||
| private static String getSessionIdForApp(BasicCookieStore cookieStore, UndertowServer server) { | ||
| return cookieStore.getCookies().stream().filter(cookie -> cookie.getName().equals("JSESSIONID") && cookie.getPath().equals(server.getContextRoot())).findAny().get().getValue(); |
There was a problem hiding this comment.
It is not part of our requirements at the moment, but I am finding as screens get wider restricting code to a reasonable line length becomes more relevant again as now it is possible to open multiple columns of source on a single screen.
There was a problem hiding this comment.
Good point, I've adjusted the line length.
fjuma
force-pushed
the
ELYWEB-222-1.9.x
branch
from
1ee6a25 to
9907ab7
Compare
fjuma
force-pushed
the
ELYWEB-222-1.9.x
branch
from
9907ab7 to
392846d
Compare
|
I have released Elytron 1.15.22.Final and confirmed that the tests pass locally. CI is still failing here because it's not able to download the newly released artifacts. I'm going to go ahead and merge to carry on with the Elytron Web release. |
https://issues.redhat.com/browse/ELYWEB-222
Depends on wildfly-security/wildfly-elytron#2094
Test case for ELY-2589
Note that the new test will fail until an Elytron component upgrade that contains wildfly-security/wildfly-elytron#2094 is included here.