Merge remote-tracking branch 'origin/master' into lforst-next-auto-tunnel

Author: Luca Forstner

.craft.yml

@@ -21,9 +21,7 @@ targets:
           cacheControl: 'public, max-age=31536000'
   - name: github
     includeNames: /^sentry-.*$/
-    excludeNames: /^sentry-replay-.*$/
   - name: npm
-    excludeNames: /^sentry-replay-.*$/
   - name: registry
     sdks:
       'npm:@sentry/browser':

.github/workflows/build.yml

@@ -22,7 +22,7 @@ env:
 
   CACHED_DEPENDENCY_PATHS: |
     ${{ github.workspace }}/node_modules
-    ${{ github.workspace }}/packages/**/node_modules
+    ${{ github.workspace }}/packages/*/node_modules
     ~/.cache/ms-playwright/
     ~/.cache/mongodb-binaries/
 
@@ -44,7 +44,7 @@ env:
 jobs:
   job_get_metadata:
     name: Get Metadata
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-20.04
     permissions:
       pull-requests: read
     steps:
@@ -121,7 +121,7 @@ jobs:
   job_install_deps:
     name: Install Dependencies
     needs: job_get_metadata
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-20.04
     timeout-minutes: 15
     steps:
       - name: 'Check out current commit (${{ needs.job_get_metadata.outputs.commit_label }})'
@@ -150,7 +150,7 @@ jobs:
   job_build:
     name: Build
     needs: [job_get_metadata, job_install_deps]
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-20.04
     timeout-minutes: 20
     steps:
       - name: Check out current commit (${{ needs.job_get_metadata.outputs.commit_label }})
@@ -187,7 +187,7 @@ jobs:
     needs: [job_get_metadata, job_build]
     # only upload the zipped layer file if we're about to release
     if: startsWith(github.ref, 'refs/heads/release/')
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-20.04
     steps:
       - name: Check out current commit (${{ needs.job_get_metadata.outputs.commit_label }})
         uses: actions/checkout@v3
@@ -226,7 +226,7 @@ jobs:
     name: Size Check
     needs: [job_get_metadata, job_build]
     timeout-minutes: 15
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-20.04
     # Size Check will error out outside of the context of a PR
     if: github.event_name == 'pull_request' || github.ref == 'refs/heads/master'
     steps:
@@ -265,7 +265,7 @@ jobs:
     # inter-package dependencies resolve cleanly.
     needs: [job_get_metadata, job_build]
     timeout-minutes: 10
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-20.04
     steps:
       - name: Check out current commit (${{ needs.job_get_metadata.outputs.commit_label }})
         uses: actions/checkout@v3
@@ -290,7 +290,7 @@ jobs:
     name: Circular Dependency Check
     needs: [job_get_metadata, job_build]
     timeout-minutes: 10
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-20.04
     steps:
       - name: Check out current commit (${{ needs.job_get_metadata.outputs.commit_label }})
         uses: actions/checkout@v3
@@ -314,7 +314,7 @@ jobs:
   job_artifacts:
     name: Upload Artifacts
     needs: [job_get_metadata, job_build]
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-20.04
     # Build artifacts are only needed for releasing workflow.
     if: startsWith(github.ref, 'refs/heads/release/')
     steps:
@@ -344,13 +344,14 @@ jobs:
             ${{ github.workspace }}/packages/browser/build/bundles/**
             ${{ github.workspace }}/packages/integrations/build/bundles/**
             ${{ github.workspace }}/packages/tracing/build/bundles/**
+            ${{ github.workspace }}/packages/replay/build/bundles/**
             ${{ github.workspace }}/packages/**/*.tgz
 
   job_unit_test:
     name: Test (Node ${{ matrix.node }})
     needs: [job_get_metadata, job_build]
     timeout-minutes: 30
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-20.04
     strategy:
       fail-fast: false
       matrix:
@@ -388,7 +389,7 @@ jobs:
     needs: [job_get_metadata, job_build]
     if: needs.job_get_metadata.outputs.changed_nextjs == 'true' || github.event_name != 'pull_request'
     timeout-minutes: 30
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-20.04
     strategy:
       fail-fast: false
       matrix:
@@ -426,7 +427,7 @@ jobs:
     needs: [job_get_metadata, job_build]
     if: needs.job_get_metadata.outputs.changed_ember == 'true' || github.event_name != 'pull_request'
     timeout-minutes: 10
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-20.04
     strategy:
       fail-fast: false
       matrix:
@@ -467,7 +468,7 @@ jobs:
     name: Playwright - ${{ (matrix.tracing_only && 'Browser + Tracing') || 'Browser' }} (${{ matrix.bundle }})
     needs: [job_get_metadata, job_build]
     if: needs.job_get_metadata.outputs.changed_browser_integration == 'true' || github.event_name != 'pull_request'
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-20.04
     strategy:
       matrix:
         bundle:
@@ -517,7 +518,7 @@ jobs:
     name: Old Browser Integration Tests (${{ matrix.browser }})
     needs: [job_get_metadata, job_build]
     if: needs.job_get_metadata.outputs.changed_browser == 'true' || github.event_name != 'pull_request'
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-20.04
     timeout-minutes: 10
     strategy:
       fail-fast: false
@@ -554,7 +555,7 @@ jobs:
   job_browser_build_tests:
     name: Browser Build Tests
     needs: [job_get_metadata, job_build]
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-20.04
     timeout-minutes: 5
     steps:
       - name: Check out current commit (${{ needs.job_get_metadata.outputs.commit_label }})
@@ -586,7 +587,7 @@ jobs:
     name: Node SDK Integration Tests (${{ matrix.node }})
     needs: [job_get_metadata, job_build]
     if: needs.job_get_metadata.outputs.changed_node == 'true' || github.event_name != 'pull_request'
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-20.04
     timeout-minutes: 10
     strategy:
       fail-fast: false
@@ -622,7 +623,7 @@ jobs:
     name: Remix SDK Integration Tests (${{ matrix.node }})
     needs: [job_get_metadata, job_build]
     if: needs.job_get_metadata.outputs.changed_remix == 'true' || github.event_name != 'pull_request'
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-20.04
     timeout-minutes: 10
     strategy:
       fail-fast: false
@@ -662,7 +663,7 @@ jobs:
       (github.event_name != 'pull_request' || github.event.pull_request.head.repo.full_name == github.repository) &&
       github.actor != 'dependabot[bot]'
     needs: [job_get_metadata, job_build]
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-20.04
     timeout-minutes: 10
     steps:
       - name: Check out current commit (${{ needs.job_get_metadata.outputs.commit_label }})
@@ -710,7 +711,7 @@ jobs:
       ]
     # Always run this, even if a dependent job failed
     if: always()
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-20.04
     steps:
       - name: Check for failures
         if: contains(needs.*.result, 'failure')

.github/workflows/canary.yml

@@ -19,7 +19,7 @@ permissions:
 jobs:
   job_canary_test:
     name: Run Canary Tests
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-20.04
     timeout-minutes: 30
     steps:
       - name: 'Check out current commit'

.github/workflows/codeql-analysis.yml

@@ -29,7 +29,7 @@ concurrency:
 jobs:
   analyze:
     name: Analyze
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-20.04
 
     strategy:
       fail-fast: false

.github/workflows/enforce-license-compliance.yml

@@ -8,7 +8,7 @@ on:
 
 jobs:
   enforce-license-compliance:
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-20.04
     steps:
       - name: 'Enforce License Compliance'
         uses: getsentry/action-enforce-license-compliance@main

.github/workflows/release.yml

@@ -13,7 +13,7 @@ on:
         required: false
 jobs:
   release:
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-20.04
     name: 'Release a new version'
     steps:
       - uses: actions/checkout@v3

.github/workflows/stale.yml

@@ -5,21 +5,21 @@ on:
   workflow_dispatch:
 jobs:
   stale:
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-20.04
     steps:
       - uses: actions/stale@87c2b794b9b47a9bec68ae03c01aeb572ffebdb1
         with:
           repo-token: ${{ github.token }}
           days-before-stale: 21
           days-before-close: 7
-          only-labels: ""
+          only-labels: ''
           operations-per-run: 100
           remove-stale-when-updated: true
           debug-only: false
           ascending: false
 
-          exempt-issue-labels: "Status: Backlog,Status: In Progress"
-          stale-issue-label: "Status: Stale"
+          exempt-issue-labels: 'Status: Backlog,Status: In Progress'
+          stale-issue-label: 'Status: Stale'
           stale-issue-message: |-
             This issue has gone three weeks without activity. In another week, I will close it.
 
@@ -29,11 +29,11 @@ jobs:
 
             "A weed is but an unloved flower." ― _Ella Wheeler Wilcox_ 🥀
           skip-stale-issue-message: false
-          close-issue-label: ""
-          close-issue-message: ""
+          close-issue-label: ''
+          close-issue-message: ''
 
-          exempt-pr-labels: "Status: Backlog,Status: In Progress"
-          stale-pr-label: "Status: Stale"
+          exempt-pr-labels: 'Status: Backlog,Status: In Progress'
+          stale-pr-label: 'Status: Stale'
           stale-pr-message: |-
             This pull request has gone three weeks without activity. In another week, I will close it.
 
@@ -44,4 +44,4 @@ jobs:
             "A weed is but an unloved flower." ― _Ella Wheeler Wilcox_ 🥀
           skip-stale-pr-message: false
           close-pr-label:
-          close-pr-message: ""
+          close-pr-message: ''

.vscode/settings.json

@@ -10,7 +10,9 @@
   "search.exclude": {
     "**/node_modules/": true,
     "**/build/": true,
-    "**/dist/": true
+    "**/dist/": true,
+    "**/coverage/": true,
+    "**/yarn-error.log": true
   },
   "typescript.tsdk": "./node_modules/typescript/lib",
   "[json]": {

CHANGELOG.md

@@ -4,6 +4,46 @@
 
 - "You miss 100 percent of the chances you don't take. — Wayne Gretzky" — Michael Scott
 
+## 7.24.2
+
+- fix(replay): Add missing rrweb type declarations (#6464)
+- fix(tracing): Check for otel before loading db module (#6461)
+- fix(tracing): Deprecate and remove `reportAllChanges` option (#6456)
+- ref(replay): Extract integration to clarify public API (#6457)
+
+## 7.24.1
+
+This patch corrects an oversight on our end which caused the Sentry Replay integration CDN bundles to be ignored when uploading bundles to our CDN.
+If you want to use the Replay CDN bundles, please use version 7.24.1 or newer.
+
+- fix(react): Add type for React Router's `encodeLocation` method (#6439)
+- fix(replay): Add CDN bundle path to release artifacts (#6452)
+- fix(tracing): Instrument cursors returned from MongoDB operations. (#6437)
+- ref(angular): Extract zonejs error unwrapper into a dedicated function (#6443)
+
+Work in this release contributed by @theofidry. Thank you for your contribution!
+
+## 7.24.0
+
+This release bumps the [`@sentry/replay`](https://github.com/getsentry/sentry-javascript/blob/master/packages/replay/README.md) package from version 0.x to 7.24.0.
+Along with this version bump, we're introducing a few breaking changes.
+Take a look at the [Replay migration guide](https://github.com/getsentry/sentry-javascript/blob/master/packages/replay/MIGRATION.md) for further information.
+The Replay version bump is the result of moving the package into the Sentry JavaScript SDK monorepo which aligns the version with our other JS SDK packages.
+**Important:** If you're using Replay with version 7.24.x or newer, make sure to also upgrade your other `@sentry/*` packages to this version.
+
+- feat(browser): Support dom.maxStringLength configuration (#6311)
+- feat(nextjs): Don't init SDK on Vercel Edge Runtime (#6408)
+- feat(nextjs): Parameterize prefix loader values (#6377)
+- feat(nextjs): Support `assetPrefix` option (#6388)
+- fix(nextjs): Inject SDK in dev mode (#6368)
+- fix(nextjs): Use `basePath` for `assetPrefix` if needed (#6424)
+- fix(node): Move `profilesSampleRate` into `BaseNodeOptions` (#6409)
+- ref(nextjs): Clean up client-side integrations code (#6382)
+- ref(nextjs): Use loader for rather than webpack plugin for injecting release (#6404)
+- ref(remix): Do not fail silently if `getClientIpAddress` throws error. (#6400)
+
+Work in this release contributed by @tomgrossman and @ZachGawlik. Thank you for your contributions!
+
 ## 7.23.0
 
 - feat(browser): Add `__SENTRY_RELEASE__` magic string (#6322)

docs/new-sdk-release-checklist.md

@@ -40,6 +40,10 @@ This page serves as a checklist of what to do when releasing a new SDK for the f
     - [ ]  Ensure dependent packages are correctly set for “Determine changed packages”
     - [ ]  Ensure unit tests run correctly
 
+- [ ]  Make sure the file paths in the ["Upload Artifacts" job](https://github.com/getsentry/sentry-javascript/blob/e5c1486eed236b878f2c49d6a04be86093816ac9/.github/workflows/build.yml#L314-L349) in `build.yml` include your new artifacts.
+    - **This is especially important, if you're adding new CDN bundles!**
+    - Tarballs (*.tgz archives) should work OOTB
+
 ## Cutting the Release
 
 When you’re ready to make the first release, there are a couple of steps that need to be performed in the **correct order**. Note that you can prepare the PRs at any time but the **merging oder** is important:

lerna.json

@@ -1,6 +1,6 @@
 {
   "lerna": "3.4.0",
-  "version": "7.23.0",
+  "version": "7.24.2",
   "packages": "packages/*",
   "npmClient": "yarn",
   "useWorkspaces": true

package.json

@@ -25,7 +25,8 @@
     "lint:eslint": "lerna run --parallel lint:eslint",
     "postpublish": "lerna run --stream --concurrency 1 postpublish",
     "test": "lerna run --ignore @sentry-internal/browser-integration-tests --ignore @sentry-internal/node-integration-tests --stream --concurrency 1 --sort test",
-    "test-ci": "ts-node ./scripts/test.ts"
+    "test-ci": "ts-node ./scripts/test.ts",
+    "postinstall": "patch-package"
   },
   "volta": {
     "node": "16.18.1",
@@ -89,6 +90,7 @@
     "mocha": "^6.1.4",
     "nodemon": "^2.0.16",
     "npm-run-all": "^4.1.5",
+    "patch-package": "^6.5.0",
     "prettier": "2.7.1",
     "recast": "^0.20.5",
     "replace-in-file": "^4.0.0",

packages/angular/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sentry/angular",
-  "version": "7.23.0",
+  "version": "7.24.2",
   "description": "Official Sentry SDK for Angular",
   "repository": "git://github.com/getsentry/sentry-javascript.git",
   "homepage": "https://github.com/getsentry/sentry-javascript/tree/master/packages/angular",
@@ -21,9 +21,9 @@
     "rxjs": "^6.5.5 || ^7.x"
   },
   "dependencies": {
-    "@sentry/browser": "7.23.0",
-    "@sentry/types": "7.23.0",
-    "@sentry/utils": "7.23.0",
+    "@sentry/browser": "7.24.2",
+    "@sentry/types": "7.24.2",
+    "@sentry/utils": "7.24.2",
     "tslib": "^2.0.0"
   },
   "devDependencies": {