Skip to content

meta(changelog): Update changelog for 8.23.0 #13228

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 27 commits into from
Aug 5, 2024
Merged

Conversation

lforst
Copy link
Contributor

@lforst lforst commented Aug 5, 2024

No description provided.

mydea and others added 26 commits August 2, 2024 13:52
Reverting the change in this PR:
#13138
For the time being (until this is debugged and fixed), this change is
reverted and the Nuxt module will still be built with the
nuxt-module-builder.

The changed build-setup led to an error in `nuxt dev` and the navigation
in the E2E test apps did not work. The error message:
```
ERROR  [worker reload] [worker init] Package import specifier "#internal/nitro/virtual/app-config" is not defined in package /Users/sigridh/Documents/DEV/sentry-javascript-examples/node_modules/nitropack/package.json imported from /Users/sigridh/Documents/DEV/sentry-javascript-examples/node_modules/nitropack/dist/runtime/config.mjs
```

Similar errors were reported here:

- nuxt/content#2736
- nuxt/icon#204
- nuxt/nuxt#13801
Add  a `getTraceData` function to the `@sentry/core` package and
re-exports it in none-browser SDKs inheriting from it.

---------

Co-authored-by: Andrei <[email protected]>
Upgrade action to artifacts v4 (previous version did some convenient
merging for us)
Our playwright tests can be flakey caused by the build process when
generating static assets. Even though we check if file exists before
symlinking, it can at times fail because file already exists. I have not
dug/thought about why this happens -- just catch/ignore and move on.

Closes #11902
This PR adds instrumentation for the [`scheduled`
handler](https://developers.cloudflare.com/workers/runtime-apis/handlers/scheduled/)
in cloudflare workers. This is used for cron triggers.

I elected to not do automatic cron instrumentation for now.
Instead I added manual instrumentation docs to the README, this will get
copied to the sentry docs eventually.
This PR adds the external contributor to the CHANGELOG.md file, so that
they are credited for their contribution. See #13062

Co-authored-by: Lms24 <[email protected]>
…AndRenderDialog (#13203)

This PR updates `_loadAndRenderDialog` to call `dialog.close()` in the `onFormClose` and `onFormSubmitted` callbacks.
…13192)

While working on adding the cloudflare sdk to some open source projects,
I noticed that setup for the cloudflare plugin was a bit of a hassle
when you needed access to environmental variables.

This PR allows users to pass a function to `sentryPagesPlugin` that
looks like so:

```ts
handler: (context: EventPluginContext<Env, Params, Data, PluginParams>) => CloudflareOptions
```

This means that users can access the cloudflare `context` (which only
exists at the request level) to get environmental variables.

```javascript
export const onRequest = Sentry.sentryPagesPlugin(context => ({
  dsn: context.env.SENTRY_DSN,
  tracesSampleRate: 1.0,
}));
```

To make some other use cases easier, this PR also exposes the
`wrapRequestHandler` API to users.
The node integration test runner depends on event order. We cannot
assert on that.

Fixes #13177
This test is quite flakey -- most of the time it correctly has the image
as LCP, but sometimes it will report the button as LCP. I have tried
fixing it by waiting for image to load, but it did not work. I wonder if
it is a bug with our LCP reporting?
…distributed tracing (#13221)

Works by adding the Sentry middlware to your `src/middleware.ts` file:

```typescript
import { sentryBeforeResponseMiddleware } from '@sentry/solidstart/middleware';
import { createMiddleware } from '@solidjs/start/middleware';

export default createMiddleware({
  onBeforeResponse: [
    sentryBeforeResponseMiddleware(),
    // Add your other middleware handlers after `sentryBeforeResponseMiddleware`
  ],
});
```

And specifying `./src/middleware.ts` in `app.config.ts`

Closes: #12551

Co-authored-by: Lukas Stracke <[email protected]>
The file implementing the nest integration in node got a bit annoying to
work with, so splitting it up.
Nuxt is using the Vue router under the hood, but the previous condition
to detect a page load (`from.name == null && from.matched.length === 0`)
does not work with Nuxt, as `from.matched` is never empty.
@lforst lforst requested review from s1gr1d and andreiborza August 5, 2024 11:17
@lforst lforst changed the base branch from develop to master August 5, 2024 11:17
@lforst lforst requested a review from a team as a code owner August 5, 2024 11:17
@lforst lforst changed the base branch from master to develop August 5, 2024 11:20
@lforst lforst changed the base branch from develop to master August 5, 2024 11:20
Copy link
Contributor

github-actions bot commented Aug 5, 2024

size-limit report 📦

Path Size
@sentry/browser 22.46 KB (added)
@sentry/browser (incl. Tracing) 34.24 KB (added)
@sentry/browser (incl. Tracing, Replay) 70.29 KB (added)
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 63.63 KB (added)
@sentry/browser (incl. Tracing, Replay with Canvas) 74.69 KB (added)
@sentry/browser (incl. Tracing, Replay, Feedback) 87.29 KB (added)
@sentry/browser (incl. Tracing, Replay, Feedback, metrics) 89.13 KB (added)
@sentry/browser (incl. metrics) 26.76 KB (added)
@sentry/browser (incl. Feedback) 39.39 KB (added)
@sentry/browser (incl. sendFeedback) 27.08 KB (added)
@sentry/browser (incl. FeedbackAsync) 31.72 KB (added)
@sentry/react 25.22 KB (added)
@sentry/react (incl. Tracing) 37.23 KB (added)
@sentry/vue 26.6 KB (added)
@sentry/vue (incl. Tracing) 36.08 KB (added)
@sentry/svelte 22.59 KB (added)
CDN Bundle 23.65 KB (added)
CDN Bundle (incl. Tracing) 35.89 KB (added)
CDN Bundle (incl. Tracing, Replay) 70.32 KB (added)
CDN Bundle (incl. Tracing, Replay, Feedback) 75.59 KB (added)
CDN Bundle - uncompressed 69.4 KB (added)
CDN Bundle (incl. Tracing) - uncompressed 106.32 KB (added)
CDN Bundle (incl. Tracing, Replay) - uncompressed 218.17 KB (added)
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 231.08 KB (added)
@sentry/nextjs (client) 37.08 KB (added)
@sentry/sveltekit (client) 34.81 KB (added)
@sentry/node 114.77 KB (added)
@sentry/node - without tracing 89.33 KB (added)
@sentry/aws-serverless 98.5 KB (added)

@lforst lforst merged commit caf8761 into master Aug 5, 2024
148 checks passed
@lforst lforst deleted the prepare-release/8.23.0 branch August 5, 2024 12:42
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.