search

wso2-extensions / identity-outbound-auth-samlsso

Apache License 2.0
5 stars 117 forks source link

Fix saml/slo url path #177

Closed piraveena closed 6 months ago

piraveena commented 6 months ago

Issue: Fixes https://github.com/wso2/product-is/issues/20275

Before this fix: Since slo URL is given as identity/saml/slo, it fails to append the / in the commonauth endpoint. It redirects to https://wso2.is.com/t/carbon.superidentity/saml/slo?sessionDataKey=9eb7504d-83fd-4414-90d1-d7d786ee86dc endpoint and fails.

Screenshot 2024-04-26 at 10 16 59

SAML tracer:

Screenshot 2024-04-26 at 10 18 58

After this fix:

Screenshot 2024-04-26 at 10 15 32

GET https://wso2.is.com/t/carbon.super/identity/saml/slo?sessionDataKey=da6a2a49-523e-42e8-97fd-7c3207e67f5c HTTP/1.1

CLAassistant commented 6 months ago

CLA assistant check
All committers have signed the CLA.