From 3e2c72dfed216b6fdd1e7f3bbad70287b1784ab3 Mon Sep 17 00:00:00 2001 From: Lucas Manuel Rodriguez Date: Thu, 10 Jul 2025 17:55:09 -0300 Subject: [PATCH] Fix ProtocolBinding attribute in SAML AuthnRequests (#30751) Fix for #30750. Using the proper values defined in: https://github.com/crewjam/saml/blob/346540312f721498fc75e69637d9250dd89f230b/metadata.go#L12-L16 --- server/sso/authorization_request.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/server/sso/authorization_request.go b/server/sso/authorization_request.go index c8224f64a2..5e1821698c 100644 --- a/server/sso/authorization_request.go +++ b/server/sso/authorization_request.go @@ -42,8 +42,8 @@ func CreateAuthorizationRequest( } samlAuthRequest, err := samlProvider.MakeAuthenticationRequest( idpURL, - "HTTPRedirectBinding", - "HTTPPostBinding", + saml.HTTPRedirectBinding, + saml.HTTPPostBinding, ) if err != nil { return "", "", ctxerr.Wrap(ctx, err, "make auth request")