apollo-usage-report - get missing operationName from the OperationDefinitionNode

[kroupacz]

Description

If someone send GraphQL query without operationName in request body, we should try to get it from the OperationDefinitionNode after the query parsing is finished.

I added this logic to the onParse (onParseEnd) handler, but I'm not sure if all the logic of the onEnveloped handler should be moved to the onParseEnd handler?

• e.g. the operationName is missing if someone send request with body

  {
  "query": "query TestQueryName{me{userAccount{id loginName}}}"
  }

  and the result in Apollo Studio metrics is: ❌

  

• if someone send request with body

  {
  "query": "query TestQueryName{me{userAccount{id loginName}}}",
  "operationName": "TestQueryName"
  }

  the result in Apollo Studio metrics is: ✅

  

[changeset-bot[bot]]

🦋 Changeset detected

Latest commit: 896bea9fd758c5619cb622c95bdbe4bd599a532e

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

This PR includes changesets to release 1 package

| Name | Type | | ---------------------------------------- | ----- | | @graphql-yoga/plugin-apollo-usage-report | 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

[kroupacz]

@EmrysMyrddin or @ardatan 🙏