Skip to content

feat: adds relay_state to SAMLIdPServiceProviderSpecV1#39276

Merged
flyinghermit merged 6 commits intomasterfrom
sshah/saml-idp-relay-state
Mar 15, 2024
Merged

feat: adds relay_state to SAMLIdPServiceProviderSpecV1#39276
flyinghermit merged 6 commits intomasterfrom
sshah/saml-idp-relay-state

Conversation

@flyinghermit
Copy link
Copy Markdown
Contributor

@flyinghermit flyinghermit commented Mar 12, 2024

This PR:

  • Adds configurable relay_state field to the SAMLIdPServiceProviderSpecV1.
  • Defines a default relay state for supported GCPWorkforce service provider preset. The default value will be used if the preset type is specified but the relay_state field is empty.

During SAML authentication process, the value of relay_state will be added to SAML IdP initiated login as relay_state HTTP parameter.

Manually tested by signing into with GCP Workforce Identity Federation with IdP initiated flow.

Issue #39076

changelog: SAML IdP service provider spec now supports configurable relay_state field. This field can be used to define relay_state parameter for IdP initiated SAML authentication flow. For supported preset service provider types, a default value will be applied if the field is not configured.

@public-teleport-github-review-bot public-teleport-github-review-bot Bot removed the request for review from strideynet March 15, 2024 13:19
@flyinghermit flyinghermit added this pull request to the merge queue Mar 15, 2024
Merged via the queue into master with commit 09bf5c3 Mar 15, 2024
@flyinghermit flyinghermit deleted the sshah/saml-idp-relay-state branch March 15, 2024 14:12
@public-teleport-github-review-bot
Copy link
Copy Markdown

@flyinghermit See the table below for backport results.

Branch Result
branch/v15 Failed

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

Successfully merging this pull request may close these issues.

3 participants