Skip to content

Add RetrieveAssertionInfoWithResponse - #236

Open
mitar wants to merge 1 commit into
russellhaering:mainfrom
mitar:retrieve-and-response
Open

Add RetrieveAssertionInfoWithResponse#236
mitar wants to merge 1 commit into
russellhaering:mainfrom
mitar:retrieve-and-response

Conversation

@mitar

@mitar mitar commented Aug 26, 2025

Copy link
Copy Markdown

Fixes #235.

This adds RetrieveAssertionInfoWithResponse, which is the same as RetrieveAssertionInfo but it also returns response itself. This is useful if you want to inspect response as well (e.g., to check InResponseTo field to validate the response).

@russellhaering Please check it out. It is small and simple.

@mitar

mitar commented Aug 28, 2025

Copy link
Copy Markdown
Author

Another use case is to be able to access response.Status to be able to log the reason why request failed.

@mitar

mitar commented Sep 22, 2025

Copy link
Copy Markdown
Author

OK, the status one can obtain from ErrInvalidValue error.

@mitar mitar mentioned this pull request Sep 23, 2025
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.

A variation of RetrieveAssertionInfo which returns also the validated response

1 participant