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

feat(protocol): Add sourcemap debug image type to protocol #1869

Merged
merged 4 commits into from
Feb 23, 2023

Conversation

mitsuhiko
Copy link
Member

@mitsuhiko mitsuhiko commented Feb 22, 2023

This adds the new sourcemap image type to the protocol so that it's possible to associate entries in the stack trace with debug files. Like abs_path we are exempting this from PII stripping so that they are not destroyed in the process for mapping purposes.

{
  "code_file": "https://mycdn.invalid/foo.js.min",
  "debug_id": "971f98e5-ce60-41ff-b2d7-235bbeb34578",
  "debug_file": "https://mycdn.invalid/foo.js.map",
  "other": "value",
  "type": "sourcemap"
}

Refs getsentry/sentry#44885

Resolves #1846

@mitsuhiko mitsuhiko requested a review from a team February 22, 2023 13:25
@jan-auer jan-auer enabled auto-merge (squash) February 23, 2023 09:16
@jan-auer jan-auer merged commit 7cb05e0 into master Feb 23, 2023
@jan-auer jan-auer deleted the feature/add-sourcemap-image-type branch February 23, 2023 09:35
jan-auer added a commit that referenced this pull request Feb 23, 2023
* master:
  feat(protocol): Add sourcemap debug image type to protocol (#1869)
  ref(statsd): Revert back the adition of metric names as tag on Sentry errors (#1873)
  feat(profiling): Add PHP support (#1871)
  fix(panic): revert sentry-types to 0.20.1 (#1872)
jan-auer added a commit that referenced this pull request Feb 23, 2023
* master:
  feat(metrics): Tag the sample decision on count_per_root_project (#1870)
  feat(protocol): Add sourcemap debug image type to protocol (#1869)
  ref(statsd): Revert back the adition of metric names as tag on Sentry errors (#1873)
  feat(profiling): Add PHP support (#1871)
  fix(panic): revert sentry-types to 0.20.1 (#1872)
  ref(server): Use async/await in all endpoints (#1862)
  ref: Buffer envelopes for broken project states (#1856)
  meta: Remove accidentally added GeoIP file (#1866)
@lforst
Copy link
Member

lforst commented Feb 23, 2023

When will this be deployed?

@jjbayer
Copy link
Member

jjbayer commented Feb 24, 2023

When will this be deployed?

@lforst : https://freight.getsentry.net/deploys/relay-pop/production/307

@lforst
Copy link
Member

lforst commented Feb 24, 2023

When will this be deployed?

@lforst : freight.getsentry.net/deploys/relay-pop/production/307

Awesome! Thank you! :)

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.

Add sourcemap image to Protocol
4 participants