From f3656b4991d050404f1ef861b4028f2953c022fc Mon Sep 17 00:00:00 2001 From: Josh Cummings <3627351+jzheaux@users.noreply.github.com> Date: Mon, 2 Feb 2026 17:11:54 -0700 Subject: [PATCH] Ensure saml_request in Tests Issue gh-17823 Signed-off-by: Josh Cummings <3627351+jzheaux@users.noreply.github.com> --- .../logout/OpenSaml5LogoutResponseValidatorTests.java | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/saml2/saml2-service-provider/src/opensaml5Test/java/org/springframework/security/saml2/provider/service/authentication/logout/OpenSaml5LogoutResponseValidatorTests.java b/saml2/saml2-service-provider/src/opensaml5Test/java/org/springframework/security/saml2/provider/service/authentication/logout/OpenSaml5LogoutResponseValidatorTests.java index b5b04995e1..e5f4ea436c 100644 --- a/saml2/saml2-service-provider/src/opensaml5Test/java/org/springframework/security/saml2/provider/service/authentication/logout/OpenSaml5LogoutResponseValidatorTests.java +++ b/saml2/saml2-service-provider/src/opensaml5Test/java/org/springframework/security/saml2/provider/service/authentication/logout/OpenSaml5LogoutResponseValidatorTests.java @@ -51,6 +51,7 @@ public class OpenSaml5LogoutResponseValidatorTests { RelyingPartyRegistration registration = signing(verifying(registration())).build(); Saml2LogoutRequest logoutRequest = Saml2LogoutRequest.withRelyingPartyRegistration(registration) .id("id") + .samlRequest("logout_request") .build(); LogoutResponse logoutResponse = TestOpenSamlObjects.assertingPartyLogoutResponse(registration); sign(logoutResponse, registration); @@ -67,6 +68,7 @@ public class OpenSaml5LogoutResponseValidatorTests { .build(); Saml2LogoutRequest logoutRequest = Saml2LogoutRequest.withRelyingPartyRegistration(registration) .id("id") + .samlRequest("logout_request") .build(); LogoutResponse logoutResponse = TestOpenSamlObjects.assertingPartyLogoutResponse(registration); Saml2LogoutResponse response = redirect(logoutResponse, registration, @@ -81,6 +83,7 @@ public class OpenSaml5LogoutResponseValidatorTests { RelyingPartyRegistration registration = registration().build(); Saml2LogoutRequest logoutRequest = Saml2LogoutRequest.withRelyingPartyRegistration(registration) .id("id") + .samlRequest("logout_request") .build(); LogoutResponse logoutResponse = TestOpenSamlObjects.assertingPartyLogoutResponse(registration); logoutResponse.getIssuer().setValue("wrong"); @@ -98,6 +101,7 @@ public class OpenSaml5LogoutResponseValidatorTests { RelyingPartyRegistration registration = registration().build(); Saml2LogoutRequest logoutRequest = Saml2LogoutRequest.withRelyingPartyRegistration(registration) .id("id") + .samlRequest("logout_request") .build(); LogoutResponse logoutResponse = TestOpenSamlObjects.assertingPartyLogoutResponse(registration); logoutResponse.setDestination("wrong"); @@ -115,6 +119,7 @@ public class OpenSaml5LogoutResponseValidatorTests { RelyingPartyRegistration registration = registration().build(); Saml2LogoutRequest logoutRequest = Saml2LogoutRequest.withRelyingPartyRegistration(registration) .id("id") + .samlRequest("logout_request") .build(); LogoutResponse logoutResponse = TestOpenSamlObjects.assertingPartyLogoutResponse(registration); logoutResponse.getStatus().getStatusCode().setValue(StatusCode.UNKNOWN_PRINCIPAL); @@ -133,6 +138,7 @@ public class OpenSaml5LogoutResponseValidatorTests { RelyingPartyRegistration registration = signing(verifying(registration())).build(); Saml2LogoutRequest logoutRequest = Saml2LogoutRequest.withRelyingPartyRegistration(registration) .id("id") + .samlRequest("logout_request") .build(); LogoutResponse logoutResponse = TestOpenSamlObjects.assertingPartyLogoutResponse(registration); sign(logoutResponse, registration);