Skip to content

meta(changelog): Update changelog for 9.20.0 #16312

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

Merged
merged 14 commits into from
May 19, 2025
Merged

Conversation

andreiborza
Copy link
Member

No description provided.

mydea and others added 11 commits May 14, 2025 14:10
…16279)

This PR fixes two things about our idle spans emitted from
`browserTracingIntegration`:

1. The navigation name was sometimes incorrect. This is because we look
at `window.location.pathname` at the time when the `popstate` event is
emitted - but at this point, this may not be updated yet. So a
`navigation` transaction would possibly have the pathname of the
previous page as transaction name.
2. The request data is also possibly incorrect - this is set by
`HttpContext` integration at event processing time. However, at this
time the `window.location` data is also usually already of the following
navigation, so the pageload would often have wrong request data
associated to it. Now, we store this on the current scope at span
creation time to ensure it is actually correct.
[Gitflow] Merge master into develop
- Adds a regex for `GET /__manifest` requests to to the low quality tx
filter
- Moves the integration into the integration folder
resolves #16237

The SDK automatically instruments the `performance.measure` API, but
doesn't support `detail`, which is the way you can attach arbitrary data
to `performance.measure`. Given you can see `details` in browser
dev-tools, we should probably support it in the same way in Sentry.

https://developer.mozilla.org/en-US/docs/Web/API/Performance/measure

detail docs:
https://developer.mozilla.org/en-US/docs/Web/API/PerformanceMeasure/detail

Detail is completely arbitrary, so we have to take care before parsing
it. I have added tests accordingly.
fixes #16247

By adjusting the generic, we'll make sure that we don't erase static
fields with the `instrumentDurableObjectWithSentry` function. See an
example below:

```js
class MyDurableObjectBase extends DurableObject {
  public static readonly VERSION = '1.0.0';
}

const MyDurableObject = instrumentDurableObjectWithSentry(
  () => ({
    dsn: 'https://example.com/sentry',
    tracesSampleRate: 1.0,
  }),
  MyDurableObjectBase,
);

console.log(MyDurableObject.VERSION); // This will now work correctly
```

By moving the `DurableObjectClass` into it's own generic (`new (state:
DurableObjectState, env: E) => T`), which we named `C`, it helps
preserve the exact constructor type of the input class, including all
its static properties and methods.

This was previously being lost by not aligning the `DurableObjectClass`
with the function return value.
Adds `maxIncomingRequestBodySize` to the Node `httpIntegration`.
The setting controls the maximum size of HTTP request bodies attached to
events.

There is the option `maxRequestBodySize`
([docs](https://develop.sentry.dev/sdk/expected-features/#attaching-request-body-in-server-sdks))
in other SDKs, but to be more specific, this is named with `incoming`.

Available options:
- 'none': No request bodies will be attached
- 'small': Request bodies up to 1,000 bytes will be attached
- 'medium': Request bodies up to 10,000 bytes will be attached (default)
- 'always': Request bodies will always be attached (up to 1 MB)

closes #16179
…-scope-forking

feat(node): Fork isolation scope in tRPC middleware
@andreiborza andreiborza requested a review from a team as a code owner May 15, 2025 14:53
@andreiborza andreiborza force-pushed the prepare-release/9.20.0 branch from c6d39b9 to 3fefae6 Compare May 15, 2025 14:59
Copy link
Contributor

github-actions bot commented May 15, 2025

size-limit report 📦

Path Size % Change Change
@sentry/browser 24.04 kB added added
@sentry/browser - with treeshaking flags 23.7 kB added added
@sentry/browser (incl. Tracing) 38.46 kB added added
@sentry/browser (incl. Tracing, Replay) 76.58 kB added added
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 69.61 kB added added
@sentry/browser (incl. Tracing, Replay with Canvas) 81.35 kB added added
@sentry/browser (incl. Tracing, Replay, Feedback) 93.46 kB added added
@sentry/browser (incl. Feedback) 40.83 kB added added
@sentry/browser (incl. sendFeedback) 28.79 kB added added
@sentry/browser (incl. FeedbackAsync) 33.66 kB added added
@sentry/react 25.86 kB added added
@sentry/react (incl. Tracing) 40.48 kB added added
@sentry/vue 28.44 kB added added
@sentry/vue (incl. Tracing) 40.28 kB added added
@sentry/svelte 24.07 kB added added
CDN Bundle 25.25 kB added added
CDN Bundle (incl. Tracing) 38.54 kB added added
CDN Bundle (incl. Tracing, Replay) 74.32 kB added added
CDN Bundle (incl. Tracing, Replay, Feedback) 79.77 kB added added
CDN Bundle - uncompressed 73.68 kB added added
CDN Bundle (incl. Tracing) - uncompressed 114.08 kB added added
CDN Bundle (incl. Tracing, Replay) - uncompressed 228.05 kB added added
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 240.87 kB added added
@sentry/nextjs (client) 42.08 kB added added
@sentry/sveltekit (client) 38.95 kB added added
@sentry/node 158.16 kB added added
@sentry/node - without tracing 98.07 kB added added
@sentry/aws-serverless 123.41 kB added added

…baggage) (#16305)

Adds the organization ID to the DSC (dynamic sampling context). This
will add the org ID as `sentry-org_id` to the baggage.

This org ID is parsed from the DSN. With the `orgId` option it's
possible to overwrite the automatically parsed organization ID.

closes #16290
Copy link
Member

@s1gr1d s1gr1d left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good!

@andreiborza andreiborza force-pushed the prepare-release/9.20.0 branch from 3fefae6 to 6139041 Compare May 16, 2025 11:04
andreiborza and others added 2 commits May 19, 2025 12:55
The utilities around wrapping are no longer in use since the switch to
diagnostics channels in #16177.
@andreiborza andreiborza force-pushed the prepare-release/9.20.0 branch from 6139041 to 6da8816 Compare May 19, 2025 10:57
@andreiborza andreiborza merged commit dac1e80 into master May 19, 2025
155 checks passed
@andreiborza andreiborza deleted the prepare-release/9.20.0 branch May 19, 2025 11:13
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.

5 participants