-
Notifications
You must be signed in to change notification settings - Fork 0
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
Set HTTP client breadcrumbs log level based on response status code #100
Comments
@cleptric would it make sense to put 4xx as error too? Is there a deeper thought putting it as warn? |
@cleptric, shouldn't we also add level |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
We should align the log level for HTTP client requests and make sure we set appropriate levels based on the response status code.
5xx should be
error
4xx should be
warning
Stakeholder(s)
@lforst @cleptric
SDKs
SDKs
The text was updated successfully, but these errors were encountered: