Skip to content

[v13] fix saml IdP service provider validation#36659

Merged
rosstimothy merged 2 commits intobranch/v13from
fspmarshall/fix-saml-validation-v13
Jan 18, 2024
Merged

[v13] fix saml IdP service provider validation#36659
rosstimothy merged 2 commits intobranch/v13from
fspmarshall/fix-saml-validation-v13

Conversation

@fspmarshall
Copy link
Copy Markdown
Contributor

@fspmarshall fspmarshall commented Jan 12, 2024

Backport #36057 and #36602 to branch/v13

changelog: Fixed an issue where valid saml entity descriptors could be rejected.

@github-actions github-actions Bot added backport size/sm tctl tctl - Teleport admin tool labels Jan 12, 2024
fspmarshall and others added 2 commits January 18, 2024 15:23
* - FilterSAMLEntityDescriptor only if entity descriptor is not empty.
- only issue warning on unsupported acs bindings.

* include HTTPArtifactBinding in supported acs bindings

* update TestFilterSAMLEntityDescriptor

* exclude HTTPArtifactBinding and filter them from generated entity descriptor

* remove HTTPArtifactBinding from test entity descriptor
@fspmarshall fspmarshall force-pushed the fspmarshall/fix-saml-validation-v13 branch from 159d3e8 to f16218c Compare January 18, 2024 15:23
@fspmarshall fspmarshall enabled auto-merge January 18, 2024 15:23
@fspmarshall fspmarshall added this pull request to the merge queue Jan 18, 2024
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jan 18, 2024
@rosstimothy rosstimothy added this pull request to the merge queue Jan 18, 2024
Merged via the queue into branch/v13 with commit a295748 Jan 18, 2024
@rosstimothy rosstimothy deleted the fspmarshall/fix-saml-validation-v13 branch January 18, 2024 22:46
@camscale camscale mentioned this pull request Feb 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport size/sm tctl tctl - Teleport admin tool

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants