Apollo Link onError should provide access to stack trace

[adrian-skybaker]

When handling errors in https://www.apollographql.com/docs/react/api/link/apollo-link-error/, there is no ability to determine the stack trace of the error that includes the original trigger for the query.

This is problematic both when logging errors locally, and reporting helpful errors to tools like Sentry.io, which accept Error objects and present stack traces helpfully.

Raising a new Error object from within an onError handler doesn't work, presumably because of the callback nature of the handler.

The only workaround I've found is to implement a separate link that populates the stack into the context:

new ApolloLink((operation, forward) => {
  operation.setContext({ clientStackTrace: new Error().stack });
  return forward(operation);
});

This is not ideal as it wastefully records the stack for every request, even if no error occurs.

This gap has been raised several times, but I don't believe it's currently captured as a feature request here:

• https://community.apollographql.com/t/client-stacktraces/6765
• https://community.apollographql.com/t/apollo-client-on-backend-stack-trace-never-shows-app-related-lines/1360
• https://github.com/apollographql/apollo-client/issues/3858
• https://stackoverflow.com/questions/61410494/how-do-i-make-apollo-client-tell-me-where-in-my-code-the-error-happened

[jerelmiller]

Hey @adrian-skybaker 👋

Thanks for opening this request! I think it makes sense for us to add support for Error.cause here which I would imagine would help here quite a bit. This is something we've talked about internally a couple times, but this is good signal that we should look at this perhaps sooner than later.

[durchanek]

This would be very helpful, I was also confused why Sentry doesn't show correct stack traces for GQL errors.

[eudis-allwhere]

Same issue here, hard to trace issues at the Sentry.captureExpception if its called once at the onError apolloLink level. Would be useful to see where the original call and its context came from in the stack

[jerelmiller]

Hey all 👋

As of 3.11, we are now setting the cause property on ApolloError. If you end up upgrading, let me me know how this works for you!