fix(GraphQLHandler): move cookie parsing to parse phase #1957

mattcosta7 · 2024-01-08T04:15:23Z

Moves getAllRequestCookies parsing in GraphQLHandler from extendResolverArgs to parse step.

This normalizes its usage with that of HttpHandler.

Also updates tests to account for this

src/core/handlers/GraphQLHandler.ts

kettanaito

Looks great! Thank you 🚀

kettanaito · 2024-01-12T15:58:10Z

Released: v2.0.14 🎉

This has been released in v2.0.14!

Make sure to always update to the latest version (npm i msw@latest) to get the newest features and bug fixes.

Predictable release automation by @ossjs/release.

fix(GraphQLHandler): move cookie parsing to parse phase

521bf34

mattcosta7 commented Jan 8, 2024

View reviewed changes

src/core/handlers/GraphQLHandler.ts Show resolved Hide resolved

mattcosta7 marked this pull request as ready for review January 8, 2024 04:19

mattcosta7 mentioned this pull request Jan 8, 2024

fix: additional caching for improved handler performance #1953

Closed

mattcosta7 requested a review from kettanaito January 8, 2024 15:13

mattcosta7 self-assigned this Jan 8, 2024

kettanaito approved these changes Jan 10, 2024

View reviewed changes

Merge branch 'main' into move-graphql-cookies-to-parse-phase

4527016

kettanaito added the release candidate label Jan 10, 2024

Merge branch 'main' into move-graphql-cookies-to-parse-phase

63a857c

kettanaito merged commit ba87f99 into main Jan 12, 2024
11 checks passed

kettanaito deleted the move-graphql-cookies-to-parse-phase branch January 12, 2024 14:48