Skip to content

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

Merged
jan-auer merged 4 commits intomasterfrom
feature/add-sourcemap-image-type
Feb 23, 2023
Merged

feat(protocol): Add sourcemap debug image type to protocol#1869
jan-auer merged 4 commits intomasterfrom
feature/add-sourcemap-image-type

Conversation

@mitsuhiko
Copy link
Contributor

@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
Contributor

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
Contributor

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