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

don't discard request body #440

Merged

Conversation

smaeda-ks
Copy link
Contributor

Continuing the previous PR: #434

Fastly Varnish would discard the request body when return(lookup) is called except POST method.
We can fake the request method to POST to save the request body then later we restore the original request method in vcl_miss.

current: https://fiddle.fastlydemo.net/fiddle/c9e8659c
fix: https://fiddle.fastlydemo.net/fiddle/ae94b715
* see the data field of the response body

@smaeda-ks smaeda-ks added the bug label Mar 11, 2021
@smaeda-ks smaeda-ks requested a review from vvuksan March 11, 2021 11:26
@smaeda-ks smaeda-ks self-assigned this Mar 11, 2021
@vvuksan vvuksan merged commit f0f3cd3 into fastly:master Mar 23, 2021
@vvuksan
Copy link
Contributor

vvuksan commented Mar 23, 2021

Released in 1.2.155

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants