Skip to content
Snippets Groups Projects

Catch errPrivateRepoPublicSourcegraph when resolving code views

Merged Administrator requested to merge errprivatepublic-sentry into master

Created by: lguychard

ERPRIVATEREPOPUBLICSOURCEGRAPHCOM is the top reported exception in Sentry/browser-extension, while it typically is an internal error that should be handled.

All stacks I looked at had were related to the same scenario: trying to resolve diff code views on Github PRs, where resolveDiffFileInfo() calls resolveRev() to try and resolve the head and base revisions.

To avoid undesired reporting of this exception on sentry, this PR adds a catchError() block to the codeViews observable, so that in such cases we simply ignore the code view without uncaught exceptions.

Merge request reports

Merged by avatar (Jul 4, 2025 6:55pm UTC)

Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading