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

Allow setting the value of saml2:Issuer #56

Open
candrews opened this issue Apr 13, 2018 · 3 comments
Open

Allow setting the value of saml2:Issuer #56

candrews opened this issue Apr 13, 2018 · 3 comments

Comments

@candrews
Copy link
Contributor

candrews commented Apr 13, 2018

I'd like to have a way to set the value of the saml2:Issuer in saml2p:AuthnRequest (the IdP I'm using requires it to have a specific value). However, it is currently being set the entityId and there is no way (at least, that I can see, to change it.)

@candrews candrews changed the title saml2:Issuer wrong in AuthnRequest Allow setting the value of saml2:Issuer Apr 13, 2018
@ulisesbocchio
Copy link
Owner

alright, let me dig into this and will come back to you with either an existing way for you to set it, or expose a way if it's not there.

@candrews
Copy link
Contributor Author

Looks like the IdP I'm using isn't the only one that requires a specific value for the Issuer - Azure AD also has a requirement on it: https://docs.microsoft.com/en-us/azure/active-directory/develop/active-directory-single-sign-on-protocol-reference

@candrews
Copy link
Contributor Author

A workaround is to define a bean like this:

    @Bean
    public WebSSOProfile myWebSSOProfile(){
	   return new DSLWebSSOProfileImpl() {

	    	@Override
	    	protected Issuer getIssuer(String localEntityId) {
	    		return super.getIssuer("SOME VALUE HERE");
	    	}

			@Override
			public void afterPropertiesSet() throws Exception {
				// this override can be removed after https://github.com/ulisesbocchio/spring-boot-security-saml/pull/58
			}

	    };
    }

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

No branches or pull requests

2 participants