Include GraphQL operation name in the unhandled request warning about msw

[vldslv-a]

Would it be more helpful if we instead printed a stringified request body in the message?

Yes, that would be nice! It would at least give us more information about unhandled requests, because sometimes they are hard to find.

[kettanaito]

Hi, @vldslv-a. Thanks for letting us know.

We've removed the parsing of requests from the onUnhandledRequest logic because it's expensive and everybody was paying for it, even if they aren't using GraphQL (there's no way of knowing if a request is a GraphQL request until you attempt to parse it).

I am a bit hesitant to add the parsing back, even if behind an option (not a fan of configurations in general).

Would it be more helpful if we instead printed a stringified request body in the message?

[MSW] Error: intercepted a request without a matching request handler:

  • POST https://app/graphql
    {"query":"query GetUser { ... }","variales":{...}}

Reading the request body is not as expensive and will include the GraphQL request information to help you identify the unhandled request. Let me know!

[kettanaito]

@vldslv-a, if you are interested, you can open a pull request with this change! I will help you review the code and merge it for everyone to use.

[samhoooo]

Hi @kettanaito @vldslv-a. If no one is working on this, may I make a contribution?