Skip to content

Conversation

danilopiazza
Copy link
Contributor

As suggested in issue #12810, this PR updates the documentation to reflect the correct usage of the assertingparty Spring Boot auto-configuration property instead of identityprovider.

I also believe that the same documentation examples are missing the registration property; I added them in a separate commit (but I would gladly either squash the two commits, or create a separate PR entirely).

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label May 8, 2025
@danilopiazza danilopiazza force-pushed the issue-12810-saml2-docs-identityprovider-vs-assertingparty branch from bed60ad to 5a56db3 Compare May 8, 2025 20:56
@jzheaux jzheaux self-assigned this May 13, 2025
@jzheaux jzheaux added in: docs An issue in Documentation or samples type: bug A general bug and removed status: waiting-for-triage An issue we've not yet triaged labels May 13, 2025
@jzheaux jzheaux added this to the 6.3.10 milestone May 13, 2025
@jzheaux jzheaux force-pushed the issue-12810-saml2-docs-identityprovider-vs-assertingparty branch from 5a56db3 to 772a4a8 Compare May 13, 2025 00:56
@jzheaux jzheaux changed the base branch from main to 6.3.x May 13, 2025 00:56
Spring Boot 2.7 renamed spring.security.saml2.relyingparty.registration.*.identityprovider.*
to spring.security.saml2.relyingparty.registration.*.assertingparty.*.

Closes spring-projectsgh-12810.

Signed-off-by: Danilo Piazzalunga <danilopiazza@gmail.com>
Signed-off-by: Danilo Piazzalunga <danilopiazza@gmail.com>
@jzheaux jzheaux force-pushed the issue-12810-saml2-docs-identityprovider-vs-assertingparty branch from 772a4a8 to 37f72a9 Compare May 13, 2025 00:59
@jzheaux jzheaux merged commit 27319e3 into spring-projects:6.3.x May 13, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in: docs An issue in Documentation or samples type: bug A general bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants