Skip to content

Commit 3740892

Browse files
JPeer264claude
andauthored
test(cloudflare): Port the http-server integration suites to span streaming (#24195)
Removes the `traceLifecycle: 'static'` pin from the five `suites/integrations/http-server*` suites. Nothing else changes. These suites cover request body capture, so they assert on the event the worker sends and never on a transaction. None of them sets `tracesSampleRate` either, so the pin only ever selected a trace lifecycle for a client that emits no spans in the first place. Part of #24148 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
1 parent 1693d50 commit 3740892

5 files changed

Lines changed: 0 additions & 5 deletions

File tree

‎dev-packages/cloudflare-integration-tests/suites/integrations/http-server-disabled/index.ts‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@ interface Env {
77
export default Sentry.withSentry(
88
(env: Env) => ({
99
dsn: env.SENTRY_DSN,
10-
traceLifecycle: 'static',
1110
integrations: [Sentry.httpServerIntegration({ maxRequestBodySize: 'none' })],
1211
}),
1312
{

‎dev-packages/cloudflare-integration-tests/suites/integrations/http-server-filtered/index.ts‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@ interface Env {
77
export default Sentry.withSentry(
88
(env: Env) => ({
99
dsn: env.SENTRY_DSN,
10-
traceLifecycle: 'static',
1110
integrations: integrations => integrations.filter(i => i.name !== 'HttpServer'),
1211
}),
1312
{

‎dev-packages/cloudflare-integration-tests/suites/integrations/http-server-ignore/index.ts‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@ interface Env {
77
export default Sentry.withSentry(
88
(env: Env) => ({
99
dsn: env.SENTRY_DSN,
10-
traceLifecycle: 'static',
1110
integrations: [
1211
Sentry.httpServerIntegration({
1312
ignoreRequestBody: url => url.includes('/health') || url.includes('/upload'),

‎dev-packages/cloudflare-integration-tests/suites/integrations/http-server-small/index.ts‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@ interface Env {
77
export default Sentry.withSentry(
88
(env: Env) => ({
99
dsn: env.SENTRY_DSN,
10-
traceLifecycle: 'static',
1110
integrations: [Sentry.httpServerIntegration({ maxRequestBodySize: 'small' })],
1211
}),
1312
{

‎dev-packages/cloudflare-integration-tests/suites/integrations/http-server/index.ts‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,6 @@ export default Sentry.withSentry(
88
(env: Env) => ({
99
dsn: env.SENTRY_DSN,
1010
attachStacktrace: false,
11-
traceLifecycle: 'static',
1211
}),
1312
{
1413
async fetch(request, _env, _ctx) {

0 commit comments

Comments
 (0)