Merge pull request #8726 from getsentry/prepare-release/7.61.1

meta: Update CHANGELOG for 7.61.1
getsentry · Aug 3, 2023 · 8c6ea3f · 8c6ea3f
2 parents 6930d84 + 754b082
commit 8c6ea3f
Show file tree

Hide file tree

Showing 127 changed files with 1,563 additions and 500 deletions.
diff --git a/.craft.yml b/.craft.yml
@@ -1,6 +1,11 @@
 minVersion: '0.23.1'
 changelogPolicy: simple
 preReleaseCommand: bash scripts/craft-pre-release.sh
+statusProvider:
+  name: github
+  config:
+    contexts:
+      - All required tests passed or skipped
 targets:
   # NPM Targets
   ## 1. Base Packages, node or browser SDKs depend on

diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -4,6 +4,22 @@
 
 - "You miss 100 percent of the chances you don't take. — Wayne Gretzky" — Michael Scott
 
+## 7.61.1
+
+- feat(nextjs): Add `AsyncLocalStorage` async context strategy to edge SDK (#8720)
+- fix(core): Filter internal API frames for synthetic frames (#8710)
+- fix(integrations): Capture exception if any arg to console method is an error (#8671)
+- fix(node-experimental): Update auto integration lookup & readme (#8690)
+- fix(node): Add availablility check on current hub to Node `ContextLines` integration (#8715)
+- fix(replay): Ensure buffer sessions end after capturing an error (#8713)
+- fix(replay): Ensure buffer->session switch is reliable (#8712)
+- fix(replay): Ensure we debounce flush if replay too short (#8716)
+- fix(replay): Improve capture of errorIds/traceIds (#8678)
+- fix(tracing): Set correct parent span id on fetch sentry-trace header (#8687)
+- fix(utils): Avoid `pre_context` and `context_line` overlap if frame lineno is out of bounds (#8722)
+- ref(replay): Improve status logging (#8709)
+- ref(nextjs): Allow withSentryConfig to accept async config function (#8721)
+
 ## 7.61.0
 
 ### Important Changes

diff --git a/packages/angular/src/errorhandler.ts b/packages/angular/src/errorhandler.ts
@@ -83,9 +83,11 @@ class SentryErrorHandler implements AngularErrorHandler {
   protected readonly _options: ErrorHandlerOptions;
 
   /* indicates if we already registered our the afterSendEvent handler */
-  private _registeredAfterSendEventHandler = false;
+  private _registeredAfterSendEventHandler;
 
   public constructor(@Inject('errorHandlerOptions') options?: ErrorHandlerOptions) {
+    this._registeredAfterSendEventHandler = false;
+
     this._options = {
       logErrors: true,
       ...options,

diff --git a/packages/angular/src/tracing.ts b/packages/angular/src/tracing.ts
@@ -145,11 +145,14 @@ export class TraceService implements OnDestroy {
     }),
   );
 
-  private _routingSpan: Span | null = null;
+  private _routingSpan: Span | null;
 
-  private _subscription: Subscription = new Subscription();
+  private _subscription: Subscription;
 
   public constructor(private readonly _router: Router) {
+    this._routingSpan = null;
+    this._subscription = new Subscription();
+
     this._subscription.add(this.navStart$.subscribe());
     this._subscription.add(this.resEnd$.subscribe());
     this._subscription.add(this.navEnd$.subscribe());

diff --git a/packages/browser-integration-tests/suites/replay/customEvents/template.html b/packages/browser-integration-tests/suites/replay/customEvents/template.html
@@ -4,7 +4,7 @@
     <meta charset="utf-8" />
   </head>
   <body>
-    <div role="button" id="error" class="btn btn-error" aria-label="An Error">An Error</div>
+    <div role="button" id="error" class="btn btn-error" aria-label="An Error in aria-label">An Error</div>
     <button title="Button title">
       <img id="img" alt="Alt Text" />
     </button>

diff --git a/packages/browser-integration-tests/suites/replay/customEvents/test.ts b/packages/browser-integration-tests/suites/replay/customEvents/test.ts
@@ -112,12 +112,12 @@ sentryTest(
       expect.arrayContaining([
         {
           ...expectedClickBreadcrumb,
-          message: 'body > div#error.btn.btn-error[aria-label="An Error"]',
+          message: 'body > div#error.btn.btn-error[aria-label="An Error in aria-label"]',
           data: {
             nodeId: expect.any(Number),
             node: {
               attributes: {
-                'aria-label': '** *****',
+                'aria-label': '** ***** ** **********',
                 class: 'btn btn-error',
                 id: 'error',
                 role: 'button',

diff --git a/packages/browser-integration-tests/suites/replay/privacyBlock/template.html b/packages/browser-integration-tests/suites/replay/privacyBlock/template.html
@@ -11,6 +11,9 @@
     <input placeholder="Placeholder should be masked" />
     <div title="Title should be masked">Title should be masked</div>
     <svg style="width:200px;height:200px" viewBox="0 0 80 80"><path d=""/><area /><rect /></svg>
+    <div data-sentry-block>
+      <div>This should be blocked</div>
+    </div>
     <svg style="width:200px;height:200px" viewBox="0 0 80 80" data-sentry-unblock><path d=""/><area /><rect /></svg>
     <img style="width:100px;height:100px" src="file:///none.png" />
     <img data-sentry-unblock style="width:100px;height:100px" src="file:///none.png" />

diff --git a/...wser-integration-tests/suites/replay/privacyBlock/test.ts-snapshots/privacy-chromium.json b/...wser-integration-tests/suites/replay/privacyBlock/test.ts-snapshots/privacy-chromium.json
@@ -194,6 +194,21 @@
                   "textContent": "\n    ",
                   "id": 28
                 },
+                {
+                  "type": 2,
+                  "tagName": "div",
+                  "attributes": {
+                    "rr_width": "[1250-1300]px",
+                    "rr_height": "[0-50]px"
+                  },
+                  "childNodes": [],
+                  "id": 29
+                },
+                {
+                  "type": 3,
+                  "textContent": "\n    ",
+                  "id": 30
+                },
                 {
                   "type": 2,
                   "tagName": "svg",
@@ -211,32 +226,32 @@
                       },
                       "childNodes": [],
                       "isSVG": true,
-                      "id": 30
+                      "id": 32
                     },
                     {
                       "type": 2,
                       "tagName": "area",
                       "attributes": {},
                       "childNodes": [],
                       "isSVG": true,
-                      "id": 31
+                      "id": 33
                     },
                     {
                       "type": 2,
                       "tagName": "rect",
                       "attributes": {},
                       "childNodes": [],
                       "isSVG": true,
-                      "id": 32
+                      "id": 34
                     }
                   ],
                   "isSVG": true,
-                  "id": 29
+                  "id": 31
                 },
                 {
                   "type": 3,
                   "textContent": "\n    ",
-                  "id": 33
+                  "id": 35
                 },
                 {
                   "type": 2,
@@ -246,12 +261,12 @@
                     "src": "file:///none.png"
                   },
                   "childNodes": [],
-                  "id": 34
+                  "id": 36
                 },
                 {
                   "type": 3,
                   "textContent": "\n    ",
-                  "id": 35
+                  "id": 37
                 },
                 {
                   "type": 2,
@@ -262,12 +277,12 @@
                     "src": "file:///none.png"
                   },
                   "childNodes": [],
-                  "id": 36
+                  "id": 38
                 },
                 {
                   "type": 3,
                   "textContent": "\n    ",
-                  "id": 37
+                  "id": 39
                 },
                 {
                   "type": 2,
@@ -277,12 +292,12 @@
                     "rr_height": "[0-50]px"
                   },
                   "childNodes": [],
-                  "id": 38
+                  "id": 40
                 },
                 {
                   "type": 3,
                   "textContent": "\n    ",
-                  "id": 39
+                  "id": 41
                 },
                 {
                   "type": 2,
@@ -293,17 +308,17 @@
                     "rr_height": "[0-50]px"
                   },
                   "childNodes": [],
-                  "id": 40
+                  "id": 42
                 },
                 {
                   "type": 3,
                   "textContent": "\n  ",
-                  "id": 41
+                  "id": 43
                 },
                 {
                   "type": 3,
                   "textContent": "\n\n",
-                  "id": 42
+                  "id": 44
                 }
               ],
               "id": 8

diff --git a/...rowser-integration-tests/suites/replay/privacyBlock/test.ts-snapshots/privacy-webkit.json b/...rowser-integration-tests/suites/replay/privacyBlock/test.ts-snapshots/privacy-webkit.json
@@ -194,6 +194,21 @@
                   "textContent": "\n    ",
                   "id": 28
                 },
+                {
+                  "type": 2,
+                  "tagName": "div",
+                  "attributes": {
+                    "rr_width": "[1250-1300]px",
+                    "rr_height": "[0-50]px"
+                  },
+                  "childNodes": [],
+                  "id": 29
+                },
+                {
+                  "type": 3,
+                  "textContent": "\n    ",
+                  "id": 30
+                },
                 {
                   "type": 2,
                   "tagName": "svg",
@@ -211,32 +226,32 @@
                       },
                       "childNodes": [],
                       "isSVG": true,
-                      "id": 30
+                      "id": 32
                     },
                     {
                       "type": 2,
                       "tagName": "area",
                       "attributes": {},
                       "childNodes": [],
                       "isSVG": true,
-                      "id": 31
+                      "id": 33
                     },
                     {
                       "type": 2,
                       "tagName": "rect",
                       "attributes": {},
                       "childNodes": [],
                       "isSVG": true,
-                      "id": 32
+                      "id": 34
                     }
                   ],
                   "isSVG": true,
-                  "id": 29
+                  "id": 31
                 },
                 {
                   "type": 3,
                   "textContent": "\n    ",
-                  "id": 33
+                  "id": 35
                 },
                 {
                   "type": 2,
@@ -246,12 +261,12 @@
                     "src": "file:///none.png"
                   },
                   "childNodes": [],
-                  "id": 34
+                  "id": 36
                 },
                 {
                   "type": 3,
                   "textContent": "\n    ",
-                  "id": 35
+                  "id": 37
                 },
                 {
                   "type": 2,
@@ -262,12 +277,12 @@
                     "src": "file:///none.png"
                   },
                   "childNodes": [],
-                  "id": 36
+                  "id": 38
                 },
                 {
                   "type": 3,
                   "textContent": "\n    ",
-                  "id": 37
+                  "id": 39
                 },
                 {
                   "type": 2,
@@ -277,12 +292,12 @@
                     "rr_height": "[0-50]px"
                   },
                   "childNodes": [],
-                  "id": 38
+                  "id": 40
                 },
                 {
                   "type": 3,
                   "textContent": "\n    ",
-                  "id": 39
+                  "id": 41
                 },
                 {
                   "type": 2,
@@ -293,17 +308,17 @@
                     "rr_height": "[0-50]px"
                   },
                   "childNodes": [],
-                  "id": 40
+                  "id": 42
                 },
                 {
                   "type": 3,
                   "textContent": "\n  ",
-                  "id": 41
+                  "id": 43
                 },
                 {
                   "type": 3,
                   "textContent": "\n\n",
-                  "id": 42
+                  "id": 44
                 }
               ],
               "id": 8

diff --git a/packages/browser-integration-tests/suites/replay/privacyDefault/template.html b/packages/browser-integration-tests/suites/replay/privacyDefault/template.html
@@ -9,6 +9,7 @@
     <div>This should be masked by default</div>
     <div data-sentry-unmask>This should be unmasked due to data attribute</div>
     <input placeholder="Placeholder should be masked" />
+    <input data-sentry-unmask placeholder="Placeholder can be unmasked" />
     <div title="Title should be masked">Title should be masked</div>
     <svg style="width:200px;height:200px" viewBox="0 0 80 80"><path d=""/><area /><rect /></svg>
     <svg style="width:200px;height:200px" viewBox="0 0 80 80" data-sentry-unblock><path d=""/><area /><rect /></svg>