- Notifications
You must be signed in to change notification settings - Fork126
Added info logging for response#511
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
| if a retry would violate the configured policy. | ||
| """ | ||
| logger.info(f"Received status code{status_code} for{method} request") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
When we get 4xx or 5xx, do we only get a status code, or do we also get an error message of some sort?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
@benc-db I have updated such that the error messages from the server are printed and not our custom messages
3111158 intorelease/v3.7.3Uh oh!
There was an error while loading.Please reload this page.
Description
Added logging for status code and method details for easy debugging