Skip to content

Prevent request headers from getting lost#1420

Merged
designatednerd merged 2 commits intoapollographql:mainfrom
dchohfi:dchohfi/prevent-headers-lost
Sep 29, 2020
Merged

Prevent request headers from getting lost#1420
designatednerd merged 2 commits intoapollographql:mainfrom
dchohfi:dchohfi/prevent-headers-lost

Conversation

@dchohfi
Copy link
Copy Markdown
Contributor

@dchohfi dchohfi commented Sep 29, 2020

I believe that this fixes #1419

@apollo-cla
Copy link
Copy Markdown

@dchohfi: Thank you for submitting a pull request! Before we can merge it, you'll need to sign the Apollo Contributor License Agreement here: https://contribute.apollographql.com/

Copy link
Copy Markdown
Contributor

@designatednerd designatednerd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wow, whoops. 🙃

Thanks for adding tests!

@designatednerd
Copy link
Copy Markdown
Contributor

Test failure is a known flaky test, rerunning then I'll get this merged

@designatednerd designatednerd added the bug Generally incorrect behavior label Sep 29, 2020
@designatednerd designatednerd added this to the Next Release milestone Sep 29, 2020
@designatednerd designatednerd merged commit 0f2eb0f into apollographql:main Sep 29, 2020
@dchohfi
Copy link
Copy Markdown
Contributor Author

dchohfi commented Sep 30, 2020

Hey, thank you for merging this, can we get a new tag out? This is breaking my migration.

@designatednerd
Copy link
Copy Markdown
Contributor

Yeah will try to get this out today - you can also point at the commit on main if you need it right away.

@dchohfi dchohfi deleted the dchohfi/prevent-headers-lost branch November 3, 2020 21:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Generally incorrect behavior

Projects

None yet

Development

Successfully merging this pull request may close these issues.

JSONRequest losing headers when useGETForQueries is true

4 participants