search

urql-graphql / urql

The highly customizable and versatile GraphQL client with which you add on features like normalized caching as you grow.
https://urql.dev/goto/docs
MIT License
8.54k stars 444 forks source link

fix(core): support Headers being passed in fetchOptions #3505

Closed JoviDeCroock closed 4 months ago

JoviDeCroock commented 4 months ago

Resolves #3504

Summary

This aims to solve the problem of supporting Headers I opted not to do do a instanceof check for this to support node versions that don't have this by default. The downside of doing the forEach check is that we also get the [[string, string]] HeadersInit in there which is a tad harder to support

changeset-bot[bot] commented 4 months ago

🦋 Changeset detected

Latest commit: f6ab6d9d8d7bbf59052ce3f9f3e034a37c172889

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package | Name | Type | | ---------- | ----- | | @urql/core | Patch |

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

JoviDeCroock commented 4 months ago

That would be a breaking change as that is not compatible with node-fetch environments