Skip to content

[v15] add preset field to SAMLIdPServiceProviderV1 spec#39277

Merged
flyinghermit merged 3 commits intobranch/v15from
sshah/backport-37968-and-39151
Mar 14, 2024
Merged

[v15] add preset field to SAMLIdPServiceProviderV1 spec#39277
flyinghermit merged 3 commits intobranch/v15from
sshah/backport-37968-and-39151

Conversation

@flyinghermit
Copy link
Copy Markdown
Contributor

Backport #37968 and #39151 (updates the former PR) to Branch/v15

changelog: The saml_idp_service_provider spec adds a new preset field that can be used to specify predefined SAML service provider profile.

@flyinghermit flyinghermit changed the title add preset field to SAMLIdPServiceProviderV1 spec [v15] add preset field to SAMLIdPServiceProviderV1 spec Mar 12, 2024
@github-actions github-actions Bot requested review from espadolini and r0mant March 12, 2024 23:36
@flyinghermit
Copy link
Copy Markdown
Contributor Author

Manual backport.


// validatePreset validates SAMLIdPServiceProviderV1 preset field.
// preset can be either empty or one of the supported type.
func validatePreset(preset string) bool {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR is not the place to change this, but if you happen to touch this part of the codebase in the future it might be cleaner to move validatePreset to the samlsp package IMO.

@public-teleport-github-review-bot public-teleport-github-review-bot Bot removed the request for review from r0mant March 13, 2024 21:21
@flyinghermit flyinghermit added this pull request to the merge queue Mar 13, 2024
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Mar 13, 2024
@flyinghermit flyinghermit enabled auto-merge March 13, 2024 21:45
@flyinghermit flyinghermit added this pull request to the merge queue Mar 13, 2024
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Mar 13, 2024
@flyinghermit flyinghermit enabled auto-merge March 14, 2024 13:04
@flyinghermit flyinghermit added this pull request to the merge queue Mar 14, 2024
Merged via the queue into branch/v15 with commit b61e6bb Mar 14, 2024
@flyinghermit flyinghermit deleted the sshah/backport-37968-and-39151 branch March 14, 2024 13:44
@r0mant r0mant mentioned this pull request Mar 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants