confluentinc / confluentinc/vscode

Verify we send correct error message to Sentry for GraphQL query failed

Open
#2,634 0 comments 0 reactions 0 assignees View on GitHub
cleanup observability/telemetry
Dominant language
TypeScript
Stars
34
Forks
17
Avg merge
1d 22h
Merged PRs (30d)
8

Description

Noticed a small burst of these errors in Sentry for latest release. It is likely a single user based on pattern/info. However, the message is missing so it's impossible to tell what the underlying cause is. Can we infer the message better? The `error.messages` string (which would come after `GraphQL query failed:` with some sort of explanation) is not here, but it should be since it got to [this line](https://github.com/confluentinc/vscode/blob/main/src/sidecar/sidecarHandle.ts#L532)
Sentry Issue: [VSCODE-EXTENSION-10T](https://confluent.sentry.io/issues/6881070466/?referrer=github_integration)

```
Error: GraphQL query failed:
File "../src/sidecar/sidecarHandle.ts", line 532, in SidecarHandle.query
throw new Error(errorString);
File "../src/graphql/direct.ts", line 112, in getDirectResources
response = await sidecar.query(query, connectionId, true, { id: connectionId });
File "../src/loaders/directResourceLoader.ts", line 34, in DirectResourceLoader.getEnvironmentsFromGraphQL
const environment = await getDirectResources(this.connectionId);
File "../src/loaders/cachingResourceLoader.ts", line 106, in CachingResourceLoader.doLoadCoarseResources
const environments = await this.getEnvironmentsFromGraphQL();
File "../src/loaders/cachingResourceLoader.ts", line 86, in CachingResourceLoader.ensureCoarseResourcesLoaded
await this.currentlyCoarseLoadingPromise;
...
(5 additional frame(s) were not displayed)

Error: direct connection resources: GraphQL query failed:
File "../src/errors.ts", line 174, in logError
wrappedError = new CustomError(e.name, `${message}: ${e.message}`, e);
File "../src/graphql/direct.ts", line 114, in getDirectResources
logError(error, "direct connection resources", {
File "../src/loaders/directResourceLoader.ts", line 34, in DirectResourceLoader.getEnvironmentsFromGraphQL
const environment = await getDirectResources(this.connectionId);
File "../src/loaders/cachingResourceLoader.ts", line 106, in CachingResourceLoader.doLoadCoarseResources
const environments = await this.getEnvironmentsFromGraphQL();
File "../src/loaders/cachingResourceLoader.ts", line 86, in CachingResourceLoader.ensureCoarseResourcesLoaded
await this.currentlyCoarseLoadingPromise;
...
(5 additional frame(s) were not displayed)
```

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.