Skip to content

Commit

Permalink
[HGI 404] update pinned version of sentry to 7.45.0 (#741)
Browse files Browse the repository at this point in the history
* HGI-404 version upgraded of sentry from 5.12.1 to 7.45.0

* HGI-404 change cdn settings

* Package version change for sentry

* version updated and worked on PR changes

---------

Co-authored-by: Ankit Gupta <[email protected]>
  • Loading branch information
AnkitUnthinkable and AnkitSegment authored Apr 17, 2023
1 parent da03368 commit b1e9a7a
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 10 deletions.
11 changes: 4 additions & 7 deletions integrations/sentry/lib/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -16,24 +16,22 @@ var Sentry = (module.exports = integration('Sentry')
.global('Sentry')
.option('config', '')
.option('environment', null)
.option('serverName', null)
.option('release', null)
.option('ignoreErrors', []) // still exists, but not documented on Sentry's website
.option('ignoreUrls', [])
.option('whitelistUrls', [])
.option('includePaths', []) // maps to Sentry.Integrations.RewriteFrames plugin
.option('maxMessageLength', null) // deprecated
.option('logger', null)
.option('customVersionProperty', null)
.option('debug', false)
.tag(
'sentry',
'<script src="https://browser.sentry-cdn.com/5.12.1/bundle.min.js" integrity="sha384-y+an4eARFKvjzOivf/Z7JtMJhaN6b+lLQ5oFbBbUwZNNVir39cYtkjW1r6Xjbxg3" crossorigin="anonymous"></script>'
'<script src="https://browser.sentry-cdn.com/7.45.0/bundle.min.js" integrity="sha384-eB2/mQAt3oY62hGYFXiPg18greyp8WT/GvKHlsvdYbvSxBRGEhBqEX8L7giHxzvp" crossorigin="anonymous"></script>'
)
// Sentry.Integrations.RewriteFrames plugin: https://docs.sentry.io/platforms/javascript/#rewriteframes
.tag(
'plugin',
'<script src="https://browser.sentry-cdn.com/5.12.1/rewriteframes.min.js" crossorigin="anonymous"></script>'
'<script src="https://browser.sentry-cdn.com/7.45.0/rewriteframes.min.js" integrity="sha384-m1kRQsSdJkB99lz+1ZvWWjrj0SPH0wXH8y7gvdjHAtKBP8lrLrcv9iF7fOQGL8I0" crossorigin="anonymous"></script>'
));

/**
Expand All @@ -52,9 +50,8 @@ Sentry.prototype.initialize = function() {
dsn: this.options.config,
environment: this.options.environment,
release: customRelease || this.options.release,
serverName: this.options.serverName,
whitelistUrls: this.options.whitelistUrls,
blacklistUrls: this.options.ignoreUrls,
allowUrls: this.options.whitelistUrls,
denyUrls: this.options.ignoreUrls,
// ignoreErrors still exists, but is not documented on Sentry's website
// https://github.com/getsentry/sentry-javascript/blob/master/packages/core/src/integrations/inboundfilters.ts#L12
ignoreErrors: this.options.ignoreErrors,
Expand Down
2 changes: 1 addition & 1 deletion integrations/sentry/package.json
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
{
"name": "@segment/analytics.js-integration-sentry",
"description": "The Sentry analytics.js integration.",
"version": "3.0.1",
"version": "4.0.0",
"keywords": [
"analytics.js",
"analytics.js-integration",
Expand Down
2 changes: 0 additions & 2 deletions integrations/sentry/test/index.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -61,13 +61,11 @@ describe('Sentry', function() {
.global('Sentry')
.option('config', '')
.option('environment', null)
.option('serverName', null)
.option('release', null)
.option('ignoreErrors', [])
.option('ignoreUrls', [])
.option('whitelistUrls', [])
.option('includePaths', [])
.option('maxMessageLength', null)
.option('logger', null)
.option('customVersionProperty', null)
.option('debug', false)
Expand Down

0 comments on commit b1e9a7a

Please sign in to comment.