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

entityId url encoding and MDUI #6

Open
bilias opened this issue Jan 16, 2022 · 2 comments
Open

entityId url encoding and MDUI #6

bilias opened this issue Jan 16, 2022 · 2 comments

Comments

@bilias
Copy link

bilias commented Jan 16, 2022

Hi,
I've implemented SSO for Shibboleth IdP (External) as per:
https://apereo.github.io/cas/6.4.x/integration/Shibboleth.html which works fine.

I'm also trying to implement
Displaying SAML MDUI as per same document, which does not work.

I've tracked the issue to be the URL thas comes to CAS. For instance:
https://cas.example.com/cas/login?service=https%3A%2F%2Fidp.example.com%2Fidp%2FAuthn%2FExternal%3Fconversation%3De2s1%26entityId%3Dhttps%3A%2F%2Fwww.example.com%2Fsp

If I replace entityId%3D to -> entityId=
then everything works as expected.

Is it a problem of shib-cas-authn or CAS side not being able to decode %3D correctly?

@bilias
Copy link
Author

bilias commented Jan 16, 2022

In advance this only happens with
shibcas.entityIdLocation = embed

setting it back to append fixes it.

code is:

`

if ("append".equalsIgnoreCase(entityIdLocation)) {
        parameterBuilders.add(new EntityIdParameterBuilder());
    }

if ("embed".equalsIgnoreCase(entityIdLocation)) {
        serviceUrl += (new EntityIdParameterBuilder().getParameterString(request, false));
    }

public String getParameterString(final HttpServletRequest request, final boolean encode) {

`

@pspaude
Copy link
Contributor

pspaude commented Apr 20, 2022

Unfortunately all the various log4j and spring shell bugs have forced updates to Java applications and this fell off our radar. We're hoping to look at this soon. Just wanted to let you know we haven't forgotten this and will get a feature update or a response to this. Thanks!

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

No branches or pull requests

2 participants