Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: log error responses returned by Zitadel #584

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Khayet
Copy link

@Khayet Khayet commented Dec 13, 2024

When we receive an error response on an introspection call, for example a 503, we previously returned a ParseError as the response was not valid JSON. This change adds a new error type ZitadelResponseError which handles such cases by logging the status code and body of the error response. This is a breaking change as we add a Variant to the error enum which needs to be handled by the user.

When we receive an error response on an introspection call, for
example a 503, we previously returned a ParseError as the response
was not valid JSON. This change adds a new error type
ZitadelResponseError which handles such cases by logging the
status code and body of the error response. This is a breaking
change as we add a Variant to the error enum which needs
to be handled by the user.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant