Skip to content

🩺 Log rejected SAML auth responses with error reason and relay-state context - #50

Open
Actalab wants to merge 1 commit into
mainfrom
log-rejected-saml-auth-response
Open

🩺 Log rejected SAML auth responses with error reason and relay-state context#50
Actalab wants to merge 1 commit into
mainfrom
log-rejected-saml-auth-response

Conversation

@Actalab

@Actalab Actalab commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

Adds a Logger.warning at the single choke point where consume_signin_response/1 rejects an IdP auth response, so failures surface the concrete reason (:invalid_relay_state, signature/decode errors, :invalid_idp_id, …) instead of being swallowed into a session that is dropped on the cross-site ACS POST.

Logs the error, idp_id, whether the IdP echoed a RelayState, and whether it was still in the cache — enough to tell apart a dropped/mangled RelayState from a decode/validation failure.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant