-
Notifications
You must be signed in to change notification settings - Fork 563
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
Add the encoding to the HTTPResponseError #708
Conversation
… from the backend, the content-type matches. This change only applies when using "return_error_code" or "return_error_details". Signed-off-by: Daniel Ortiz <dortiz@krakend.io>
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.
Please, fix the failing tests
Signed-off-by: Daniel Ortiz <dortiz@krakend.io>
Signed-off-by: Daniel Ortiz <dortiz@krakend.io>
Resolved issues in transport/http/client/status_test.go with DeepSource Autofix
Signed-off-by: Daniel Ortiz <dortiz@krakend.io>
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.
Looks good to me
This pull request was marked as resolved a long time ago and now has been automatically locked as there has not been any recent activity after it. You can still open a new issue and reference this link. |
When returning an error from the backend, the content-type matches.
This change only applies when using
return_error_code
orreturn_error_details
.