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

Keycloak SAML Adapter subsystem does not support Wildfly 29 #22172

Closed
2 tasks done
APUSStefanie opened this issue Aug 2, 2023 · 3 comments · Fixed by #22824
Closed
2 tasks done

Keycloak SAML Adapter subsystem does not support Wildfly 29 #22172

APUSStefanie opened this issue Aug 2, 2023 · 3 comments · Fixed by #22824
Assignees
Labels
area/adapter/saml kind/bug Categorizes a PR related to a bug
Milestone

Comments

@APUSStefanie
Copy link

APUSStefanie commented Aug 2, 2023

Before reporting an issue

Area

adapter/jee-saml

Describe the bug

I installed with the new Wildfly 29 Galleon Release the keycloak saml adapter.
The sso login works fine if I use the configuration within the WAR (keycloak-saml.xml in WEB-INF).
We have different customers and test/prod stages, so we want to use the subsystem xmlns="urn:jboss:domain:keycloak-saml:1.4" for configuring IDP and SP.

When I gave the configuration back to the subsystem (we already had this on our old wildfly 23) I got an exception during deployment.

Caused by: java.lang.NoSuchMethodError: 'org.jboss.as.controller.AttributeMarshaller org.jboss.as.controller.SimpleAttributeDefinition.getAttributeMarshaller()'
at org.keycloak.keycloak-saml-wildfly-jakarta-subsystem@22.0.1//org.keycloak.subsystem.adapter.saml.extension.KeycloakSubsystemParser.writeSps(KeycloakSubsystemParser.java:501)
at org.keycloak.keycloak-saml-wildfly-jakarta-subsystem@22.0.1//org.keycloak.subsystem.adapter.saml.extension.KeycloakAdapterConfigDeploymentProcessor.getXML(KeycloakAdapterConfigDeploymentProcessor.java:99)
at org.keycloak.keycloak-saml-wildfly-jakarta-subsystem@22.0.1//org.keycloak.subsystem.adapter.saml.extension.KeycloakAdapterConfigDeploymentProcessor.addKeycloakSamlAuthData(KeycloakAdapterConfigDeploymentProcessor.java:71)
at org.keycloak.keycloak-saml-wildfly-jakarta-subsystem@22.0.1//org.keycloak.subsystem.adapter.saml.extension.KeycloakAdapterConfigDeploymentProcessor.deploy(KeycloakAdapterConfigDeploymentProcessor.java:57)
at org.jboss.as.server@21.1.0.Final//org.jboss.as.server.deployment.DeploymentUnitPhaseService.start(DeploymentUnitPhaseService.java:182)
... 8 more

It seems in the code is a method called, which is not anymore existing in the current Wildfly 29 version.

According to the wildfly and keycloak adapter docoumentation, those two versions should be compatible.

Version

SAML Adapter 22.0.1

Expected behavior

Configuration for SAML can be done again in subsystem xmlns="urn:jboss:domain:keycloak-saml:1.4"

Actual behavior

Exception during deployment

How to Reproduce?

  • Set up Wildfly 29 with SAML
  • Use subsystem xmlns="urn:jboss:domain:keycloak-saml:1.4" for SAML configuration
  • Deploy a war

Anything else?

https://www.wildfly.org/news/2023/07/21/WildFly29-Released/
https://docs.wildfly.org/29/WildFly_Elytron_Security.html#Keycloak_SAML_Integration

@mposolda
Copy link
Contributor

@pedroigor @pskopek Adding you for triage of this

@pedroigor
Copy link
Contributor

We are compiling using 27. I don't remember what is our policy around the WF version we support when using the SAML adapter but docs suggest we are supporting the latest, as mentioned by @APUSStefanie.

If so, shall we bump the version to WF latest (e.g.: 29)?

Or perhaps we can change our policy to match WF with the latest EAP version?

@pedroigor
Copy link
Contributor

Should be fixed by #22593.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/adapter/saml kind/bug Categorizes a PR related to a bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants