envelop icon indicating copy to clipboard operation
envelop copied to clipboard

[@envelop/sentry] - Spans are not added to resolvers

Open 2snEM6 opened this issue 1 year ago • 0 comments

Issue workflow progress

Progress of the issue based on the Contributor Workflow

  • [ ] 1. The issue provides a minimal reproduction available on Stackblitz.
    • Please install the latest @envelop/* packages that you are using.
    • Please make sure the reproduction is as small as possible.
  • [ ] 2. A failing test has been provided
  • [ ] 3. A local solution has been provided
  • [ ] 4. A pull request is pending review

Describe the bug

To Reproduce Steps to reproduce the behavior:

  • Install latest version of @envelop/sentry
  • Use the following settings:
includeRawResult: false, // set to `true` in order to include the execution result in the metadata collected
includeResolverArgs: true, // set to `true` in order to include the args passed to resolvers
includeExecuteVariables: false, // set to `true` in order to include the operation variables values
renameTransaction: true,
startTransaction: true,

Expected behavior

Spans should be added for each resolver in order to track performance of them individually

Environment:

  • OS: Docker
  • NodeJS: 18
  • @envelop/* versions: latest
    • @envelop/core: ??? latest

Additional context

2snEM6 avatar Jun 05 '23 17:06 2snEM6