Skip to content
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.

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

fix: Sentry exception in the side panel should not assume all events in trace are exceptions, add Sentry SDK to API server #143

Merged
merged 5 commits into from
Dec 6, 2023

Conversation

MikeShi42
Copy link
Contributor

Sentry DSN config is mainly for us to test against our own ingestor. In case anyone runs across this you need to set SENTRY_DSN to http://YOUR_KEY_WITHOUT_DASHES@ingestor:8002/0 (you need to use ingestor, not localhost as the api server won't resolve that to the ingestor)

…in trace are exceptions, add Sentry SDK to API server
Copy link

changeset-bot bot commented Dec 5, 2023

🦋 Changeset detected

Latest commit: d6edf29

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@hyperdx/api Patch
@hyperdx/app Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@kodiakhq kodiakhq bot merged commit 3a93196 into main Dec 6, 2023
4 checks passed
@kodiakhq kodiakhq bot deleted the mikeshi/fix-sentry-exception-side-panel-rendering branch December 6, 2023 07:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants