Skip to content

Report error reason during validation#116

Merged
rmccue merged 1 commit intomasterfrom
better-error-reporting-for-invalid
Jan 15, 2026
Merged

Report error reason during validation#116
rmccue merged 1 commit intomasterfrom
better-error-reporting-for-invalid

Conversation

@rmccue
Copy link
Member

@rmccue rmccue commented Jan 15, 2026

If an error occurs during validation, the UI displays a useless message with a lack of detail:

Error: Could not parse the authentication response, please forward this error to your administrator: "invalid_response"

This PR handles the common (only?) case, where we have a single error that occurs. It adds the error "reason" to this message:

Error: Could not parse the authentication response, please forward this error to your administrator: "invalid_response" ("SAML Response must contain 1 assertion")

This aids in debugging setup of a SAML integration.

@rmccue rmccue requested a review from joehoyle January 15, 2026 16:15
@rmccue rmccue merged commit b704e11 into master Jan 15, 2026
1 check failed
@rmccue rmccue deleted the better-error-reporting-for-invalid branch January 15, 2026 17:04
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.

2 participants