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-8742] Fix tests to install Elytron provider #10157

Closed
wants to merge 1 commit into from

Conversation

dmlloyd
Copy link
Member

@dmlloyd dmlloyd commented Jun 6, 2017

No description provided.

@dmlloyd
Copy link
Member Author

dmlloyd commented Jun 6, 2017

Depends on #10156 (will probably need a re-test then)

@darranl
Copy link
Contributor

darranl commented Jun 6, 2017

Is this really needed? The AuthenticationConfiguration default should be able to discover the provider automatically.

@dmlloyd
Copy link
Member Author

dmlloyd commented Jun 6, 2017

Emperically, no, it does not seem to do so. This patch allows the test to proceed where previously it failed.

@kabir
Copy link
Contributor

kabir commented Jun 7, 2017

Retest this please

@darranl
Copy link
Contributor

darranl commented Jun 7, 2017

This is the logging I can get out of the server: -

2017-06-07 19:17:41,085 TRACE [org.jboss.remoting.remote] (default I/O-7) No message ready; returning
2017-06-07 19:17:41,618 TRACE [org.wildfly.security] (default I/O-11) Permission mapping: identity [anonymous] with roles [] implies ("org.wildfly.security.auth.permission.LoginPermission" "") = false
2017-06-07 19:17:41,629 ERROR [org.apache.activemq.artemis.core.server] (default I/O-11) AMQ224018: Failed to create session: ActiveMQSecurityException[errorType=SECURITY_EXCEPTION message=AMQ119031: Unable to validate user]
at org.apache.activemq.artemis.core.security.impl.SecurityStoreImpl.authenticate(SecurityStoreImpl.java:144)
at org.apache.activemq.artemis.core.server.impl.ActiveMQServerImpl.createSession(ActiveMQServerImpl.java:1283)
at org.apache.activemq.artemis.core.protocol.core.impl.ActiveMQPacketHandler.handleCreateSession(ActiveMQPacketHandler.java:156)

@darranl
Copy link
Contributor

darranl commented Jun 8, 2017

Based on my comments in the Jira issues I think this PR should be closed until the next Elytron version is included, at that point when the tests are re-enabled IMO some work is also required on the tests to ensure the anonymous identity is granted the LoginPermission to allow the connection to be established.

@dmlloyd
Copy link
Member Author

dmlloyd commented Jun 8, 2017

Yeah that's fine.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants