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.

Sign up for GitHub

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

Jump to bottom

Make it possible to pass all responses back to the client. #4

Open
mschout opened this issue Sep 2, 2020 · 0 comments · May be fixed by #5
Open

Make it possible to pass all responses back to the client. #4

mschout opened this issue Sep 2, 2020 · 0 comments · May be fixed by #5

Comments

@mschout
Copy link

mschout commented Sep 2, 2020

We ran into a problem with this plugin where we were proxying to an elasticsearch container making JSON requests to it where we would get 500 response with Failed to fetch data from backend insetad of the nicely formatted 400 Bad Request with JSON payload that the elasticsearch container returned.

This is caused by the Proxy plugin discarding the response and rewriting it to 500 Internal Error with plain text Failed to fetch data from backend content.

It would be nice if there was an option to make it return all responses back out, including errors.
mschout added a commit to CourtAPI/Mojolicious-Plugin-Proxy that referenced this issue Sep 2, 2020
@mschout
Add preserve_error_response option
905a8cb 
This option will pass through response errors from the backend to the
client unchanged

Fixes marcusramberg#4
@mschout mschout linked a pull request Sep 2, 2020 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add preserve_error_response option CourtAPI/Mojolicious-Plugin-Proxy
1 participant
@mschout