Skip to content

chore(auth-client): Add Sentry bread crumbs to auth-client requests#20699

Open
dschom wants to merge 1 commit into
mainfrom
add-sentry-bread-crumbs
Open

chore(auth-client): Add Sentry bread crumbs to auth-client requests#20699
dschom wants to merge 1 commit into
mainfrom
add-sentry-bread-crumbs

Conversation

@dschom

@dschom dschom commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

Because

  • We need more visibility into why down stream errors may have failed on previous fetches

This pull request

  • Adds bread crumbs to auth-client so we can better track impact of failed requests

Issue that this pull request solves

Closes: (issue number)

Checklist

Put an x in the boxes that apply

  • My commit is GPG signed.
  • If applicable, I have modified or added tests which pass locally.
  • I have added necessary documentation (if appropriate).
  • I have verified that my changes render correctly in RTL (if appropriate).
  • I have manually reviewed all AI generated code.

@dschom dschom requested a review from a team as a code owner June 4, 2026 23:23
data: {
path,
enc_path: btoa(path),
status: result.status,

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't this be response? The result variable would be undefined here and we probably have to drop the code and errno parts?

@vbudhram vbudhram left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dschom r+wc

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants