Skip to content

[SAML] Allow to override metadata file name on Saml2MetadataFilter #9317

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

Closed
GitHanter opened this issue Dec 29, 2020 · 5 comments
Closed

[SAML] Allow to override metadata file name on Saml2MetadataFilter #9317

GitHanter opened this issue Dec 29, 2020 · 5 comments
Assignees
Labels
in: saml2 An issue in SAML2 modules status: feedback-provided Feedback has been provided type: enhancement A general enhancement
Milestone

Comments

@GitHanter
Copy link
Contributor

Currently writeMetadataToResponse method on Saml2MetadataFilter is private, we may need it protected, so we can allow developer to customize the SP metadata file name.

@GitHanter GitHanter added status: waiting-for-triage An issue we've not yet triaged type: enhancement A general enhancement labels Dec 29, 2020
@jzheaux jzheaux self-assigned this Jan 5, 2021
@jzheaux jzheaux added in: saml2 An issue in SAML2 modules and removed status: waiting-for-triage An issue we've not yet triaged labels Jan 5, 2021
@jzheaux
Copy link
Contributor

jzheaux commented Jan 5, 2021

Thanks for the suggestion, @GitHanter.

How would your application determine the name of the file? I'm wondering if something more involved than Saml2MetadataFilter#setMetadataFilename is needed.

@jzheaux jzheaux added status: feedback-provided Feedback has been provided status: waiting-for-feedback We need additional information before we can continue and removed status: feedback-provided Feedback has been provided labels Jan 5, 2021
@spring-projects-issues
Copy link

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.

@spring-projects-issues spring-projects-issues added the status: feedback-reminder We've sent a reminder that we need additional information before we can continue label Jan 12, 2021
@GitHanter
Copy link
Contributor Author

GitHanter commented Jan 13, 2021

Thanks for the suggestion, @GitHanter.

How would your application determine the name of the file? I'm wondering if something more involved than Saml2MetadataFilter#setMetadataFilename is needed.

We name it as sp-<registrationId>-metadata.xml, but I think we should give developer a chance to change the metadata file name.

@spring-projects-issues spring-projects-issues added status: feedback-provided Feedback has been provided and removed status: waiting-for-feedback We need additional information before we can continue status: feedback-reminder We've sent a reminder that we need additional information before we can continue labels Jan 13, 2021
@jzheaux
Copy link
Contributor

jzheaux commented Jan 19, 2021

Thanks, @GitHanter, that makes sense.

It would be nice to have a setter that supports the {registrationId} placeholder as other SAML configuration values do.

Would you be able to provide a PR to add Saml2MetadataFilter#setMetadataFilename(String), where the filter resolves {registrationId} placeholders? Then, you could do:

Saml2MetadataFilter filter = new ...
filter.setMetadataFilename("sp-{registrationId}-metadata.xml");

@jzheaux jzheaux added status: waiting-for-feedback We need additional information before we can continue and removed status: feedback-provided Feedback has been provided labels Jan 19, 2021
@GitHanter
Copy link
Contributor Author

@jzheaux Sure, I'll do the PR this weekend.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in: saml2 An issue in SAML2 modules status: feedback-provided Feedback has been provided type: enhancement A general enhancement
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants