Does this integration follow the graphql client guidelines of sentry?

[ab-pm]

I just came across the official description of features that are commonly expected in GraphQL Client Integrations by Sentry. Does this package follow these guidelines, in particular about which properties to attach to the breadcrumb?

If yes, it would be great if that could be documented in the readme; if no, what is required to implement this?

[ab-pm]

Actually I wonder, is this whole package deprecated by getsentry/sentry-javascript#13783? (Which seems rather new and unfortunately Sentry.graphqlClientIntegration is still undocumented in https://docs.sentry.io/platforms/javascript/configuration/integrations/)

[spawnia]

This is the first time I have seen https://develop.sentry.dev/sdk/expected-features/#graphql-client-integrations. I do not know whether it existed or was considered during initial development of this package.

I noted the lack of documentation about the built-in support in getsentry/sentry-javascript#13399 (comment). From what I could infer from the pull request, there only seems to be a single configuration option for the integration that is used to identify which HTTP requests are GraphQL requests. Thus, I believe this package provides additional benefits due to the fine-grained configuration options it has.

[jh-grundfos]

Just set up Sentry using graphqlClientIntegration and it works great! I think it effectively replaces this package for anyone using Sentry for React > v9, but the package might still be useful for anyone using < 7.

It doesn't seem to work for v8, as the package calls a configureScope method in the Sentry package that is no longer present after 7

[spawnia]

It doesn't seem to work for v8, as the package calls a configureScope method in the Sentry package that is no longer present after 7

That was changed in https://github.com/DiederikvandenB/apollo-link-sentry/releases/tag/v4.0.0, it seems to me that you have been using v3.

[jh-grundfos]

You're right, I was 👍 Good catch.