-
Notifications
You must be signed in to change notification settings - Fork 17
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 support for response redirects #40
Conversation
This would be easier to implement now with recent updates in the grammar. |
oof I actually think recent changes in main broke some thing outside of this PR. in particular, external bodies are broken with response handler scripts. specifically this test fails: GET https://www.example.com
< ./user.json
> /handler.json expected result:
actual result:
it seems like additional content after an external body causes everything to just be parsed as a raw body |
0f572d4
to
3870b81
Compare
eda58c3
to
ebc29ed
Compare
@boltlessengineer I rebased on main, conflicts weren't too terrible :) |
61e6a09
to
7c68e1c
Compare
@boltlessengineer bump :) |
Merging now. Thank you for your contribution! |
as supported in jetbrains IDEs: https://jetbrains.com/help/rider/Exploring_HTTP_Syntax.html#response-redirect