[AXON-1721]-sentry-logging #1460

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Merged

teg-atlassian merged 14 commits into main from AXON-1721-sentry-logging

Jan 8, 2026

Contributor

teg-atlassian commented Jan 6, 2026 •

edited

Loading

What Is This Change?

This PR introduces Sentry logging.

It has encapulated the Sentry logging logic in sentry.ts.
We only log errors to Sentry.
Currently this logic is behind a feature flag atlascode-sentry-logging.
Sentry configuration is in .env.example. To run it locally, please copy the configuration to your local .env file

the logs in sentry can be seen at https://atlassian-2y.sentry.io/issues/errors-outages/?project=4510634894688256

Screenshot 2026-01-06 at 2 47 18 PM

How Has This Been Tested?

sentry.test.ts with result shown below

logger.test.ts with result shown below

Basic checks:

npm run lint
npm run test

Advanced checks:

If Atlassian employee & Bitbucket changes: did you test with DC in mind? See Instructions

Recommendations:

Update the CHANGELOG if making a user facing change

teg-atlassian requested review from Blastoplex, amarg-at, bhorai-atl, bwieger-atlassian-com, cabella-dot, ccallcottstevens, dchiew-atl, jwang19-atlassian, marcomura, matt-lassian, mattcolman, om-ukr, sdzh-atlassian and sspanchenko as code owners

January 6, 2026 22:49

teg-atlassian force-pushed the AXON-1721-sentry-logging branch 2 times, most recently from 33c6ece to 4c22752 Compare

January 6, 2026 22:52

teg-atlassian changed the title ~~introducing Sentry Logging~~ [AXON-1721]-sentry-logging

sdzh-atlassian reviewed

View reviewed changes

src/sentry.ts Show resolved Hide resolved

sdzh-atlassian reviewed

View reviewed changes

package.json Outdated Show resolved Hide resolved

teg-atlassian force-pushed the AXON-1721-sentry-logging branch 2 times, most recently from 920ba1b to 6a13da7 Compare

January 6, 2026 23:24

sdzh-atlassian reviewed

View reviewed changes

src/sentry.ts Outdated Show resolved Hide resolved

sdzh-atlassian reviewed

View reviewed changes

src/container.ts Outdated Show resolved Hide resolved

sdzh-atlassian reviewed

View reviewed changes

src/container.ts Outdated Show resolved Hide resolved

sdzh-atlassian reviewed

View reviewed changes

src/logger.ts Outdated Show resolved Hide resolved

teg-atlassian force-pushed the AXON-1721-sentry-logging branch 2 times, most recently from f14b4dd to 34d39a4 Compare

January 7, 2026 19:42

teg-atlassian added 3 commits

January 7, 2026 11:45


          introducing Sentry Logging

0bc1d3c


          addressing code review comments

53d0cb6


          webpack compliation issue because of sentry

d298344

teg-atlassian added 6 commits

January 7, 2026 11:45


          webpack compliation issue because of sentry

678a2fc


          added analytics event when sentry fails

920f431


          added analytics event when sentry fails

2a71468


          added analytics event when sentry fails

ae5d989


          added analytics event when sentry fails

81ed68d


          added analytics event when sentry fails

26a49b8

teg-atlassian force-pushed the AXON-1721-sentry-logging branch from 34d39a4 to 26a49b8 Compare

January 7, 2026 19:45


          removing duplicate configs

42b8a84

sdzh-atlassian reviewed

View reviewed changes

__tests__/setupTests.js Outdated Show resolved Hide resolved

teg-atlassian added 3 commits

January 7, 2026 15:01


          code review comment

a608cb4


          Merge branch 'main' into AXON-1721-sentry-logging

c00b2cd


          Merge branch 'main' into AXON-1721-sentry-logging

6386e27

jwang19-atlassian approved these changes

View reviewed changes

Contributor

jwang19-atlassian left a comment

lgtm


          Merge branch 'main' into AXON-1721-sentry-logging

9b7122e

teg-atlassian enabled auto-merge (squash)

January 8, 2026 19:11

teg-atlassian merged commit 00efab2 into main

13 checks passed

teg-atlassian deleted the AXON-1721-sentry-logging branch

January 8, 2026 19:31

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

Reviewers

sdzh-atlassian sdzh-atlassian left review comments

jwang19-atlassian jwang19-atlassian approved these changes

bwieger-atlassian-com Awaiting requested review from bwieger-atlassian-com bwieger-atlassian-com is a code owner

cabella-dot Awaiting requested review from cabella-dot cabella-dot is a code owner

marcomura Awaiting requested review from marcomura marcomura is a code owner

bhorai-atl Awaiting requested review from bhorai-atl

om-ukr Awaiting requested review from om-ukr om-ukr is a code owner

sspanchenko Awaiting requested review from sspanchenko

Blastoplex Awaiting requested review from Blastoplex Blastoplex is a code owner

mattcolman Awaiting requested review from mattcolman mattcolman is a code owner

matt-lassian Awaiting requested review from matt-lassian matt-lassian is a code owner

dchiew-atl Awaiting requested review from dchiew-atl dchiew-atl is a code owner

amarg-at Awaiting requested review from amarg-at amarg-at is a code owner

ccallcottstevens Awaiting requested review from ccallcottstevens ccallcottstevens is a code owner

Labels

None yet