sentry-javascript icon indicating copy to clipboard operation
sentry-javascript copied to clipboard

feat(remix): Migrate to `opentelemetry-instrumentation-remix`.

Open onurtemizkan opened this issue 1 year ago • 0 comments

Ref: https://github.com/getsentry/sentry-javascript/issues/11040

Migrates Remix server-side SDK to opentelemetry-instrumentation-remix.

This PR removes:

  • Performance tracing on action / loader / documentRequest functions. Leaving them to be traced by opentelemetry-instrumentation-remix
  • Request handler instrumentation as they are also traced by opentelemetry-instrumentation-remix
  • Express server adapter (This breakingly changes the DX for the better.)
  • Auto-instrumentation for http as default integration for Remix SDK.

Also:

Migrates Remix integration tests from Jest to Vitest.

  • Related issues: https://github.com/jestjs/jest/issues/15033 https://github.com/elastic/require-in-the-middle/issues/50

Fixes Backlogged Issues:

  • Fixes: #12082 - Code
  • Fixes: #9737 - Test

onurtemizkan avatar May 17 '24 14:05 onurtemizkan