-
Notifications
You must be signed in to change notification settings - Fork 347
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
Fix headers not being usable in sessionId template #5103
base: master
Are you sure you want to change the base?
Conversation
🦋 Changeset detectedLatest commit: 2e9caff The changes in this PR will be included in the next version bump. This PR includes changesets to release 4 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Hmm we shouldn't need that actually; |
c3c0b9e
to
696f9c7
Compare
72ed5cd
to
fbf068b
Compare
b78fdb4
to
a1bfc49
Compare
f6b9ba0
to
4d52840
Compare
🚨 IMPORTANT: Please do not create a Pull Request without creating an issue first.
Any change needs to be discussed before proceeding. Failure to do so may result in the rejection of
the pull request.
Description
Makes headers usable in session id when using node's fetch.
Fixes #5102
Related #4109
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
This has only been tested locally, I was not able to write a proper test case with a fetch Request object.
However you can reproduce it with the steps from #5102.
Test Environment:
@graphql-mesh/plugin-response-cache
: 0.2.7Checklist:
CONTRIBUTING doc and the
style guidelines of this project