confluentinc / confluentinc/vscode

Fix ordering between initSentry() and WriteableTmpDir.getInstance().determine()

Open Beginner friendly
#3,418 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
TypeScript
Stars
34
Forks
17
Avg merge
1d 22h
Merged PRs (30d)
8

Description

Fix the Sentry/logging bootstrap ordering so initSentry()'s first getSentryScope() call
can't fire a log write before determine() resolves. Do this through moving the initSentry() call in extension.ts to after `await WriteableTmpDir.getInstance().determine()` and place a comment as to why ("// Logging calls within initSentry() depend on first knowing a writeable tmp dir").

Contributor guide

Open the contributing guide

Research direction

Open extension.ts and inspect the initSentry() call alongside await WriteableTmpDir.getInstance().determine(). Move initSentry() after determine() and add the specified comment; done means determine() completes before initSentry() can log.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript, vscode
Domain
observability
Issue type
Bug
Difficulty
1/5
Estimated time
Under an hour
Activity status
Quiet
Clarity
Clearly specified
Newbie friendliness
90/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.