diff --git a/dev-packages/node-integration-tests/suites/tracing/httpIntegration/server-ignoreOutgoingRequests.js b/dev-packages/node-integration-tests/suites/tracing/httpIntegration/server-ignoreOutgoingRequests.js index b42fa97fab08..8c5c1472dcfa 100644 --- a/dev-packages/node-integration-tests/suites/tracing/httpIntegration/server-ignoreOutgoingRequests.js +++ b/dev-packages/node-integration-tests/suites/tracing/httpIntegration/server-ignoreOutgoingRequests.js @@ -1,6 +1,5 @@ const { loggingTransport } = require('@sentry-internal/node-integration-tests'); const Sentry = require('@sentry/node'); -const http = require('http'); Sentry.init({ dsn: 'https://public@dsn.ingest.sentry.io/1337', @@ -11,7 +10,7 @@ Sentry.init({ integrations: [ Sentry.httpIntegration({ ignoreOutgoingRequests: (url, request) => { - if (url === 'http://example.com/blockUrl') { + if (url === 'https://example.com/blockUrl') { return true; } @@ -24,6 +23,8 @@ Sentry.init({ ], }); +const https = require('https'); + // express must be required after Sentry is initialized const express = require('express'); const cors = require('cors'); @@ -34,16 +35,16 @@ const app = express(); app.use(cors()); app.get('/testUrl', (_req, response) => { - makeHttpRequest('http://example.com/blockUrl').then(() => { - makeHttpRequest('http://example.com/pass').then(() => { + makeHttpRequest('https://example.com/blockUrl').then(() => { + makeHttpRequest('https://example.com/pass').then(() => { response.send({ response: 'done' }); }); }); }); app.get('/testRequest', (_req, response) => { - makeHttpRequest('http://example.com/blockRequest').then(() => { - makeHttpRequest('http://example.com/pass').then(() => { + makeHttpRequest('https://example.com/blockRequest').then(() => { + makeHttpRequest('https://example.com/pass').then(() => { response.send({ response: 'done' }); }); }); @@ -55,7 +56,7 @@ startExpressServerAndSendPortToRunner(app); function makeHttpRequest(url) { return new Promise((resolve, reject) => { - http + https .get(url, res => { res.on('data', () => {}); res.on('end', () => { diff --git a/dev-packages/node-integration-tests/suites/tracing/httpIntegration/test.ts b/dev-packages/node-integration-tests/suites/tracing/httpIntegration/test.ts index 7fc6a5f05efa..c3d72d4708c9 100644 --- a/dev-packages/node-integration-tests/suites/tracing/httpIntegration/test.ts +++ b/dev-packages/node-integration-tests/suites/tracing/httpIntegration/test.ts @@ -137,11 +137,11 @@ describe('httpIntegration', () => { const requestSpans = event.spans?.filter(span => span.op === 'http.client'); expect(requestSpans).toHaveLength(1); - expect(requestSpans![0]?.description).toBe('GET http://example.com/pass'); + expect(requestSpans![0]?.description).toBe('GET https://example.com/pass'); const breadcrumbs = event.breadcrumbs?.filter(b => b.category === 'http'); expect(breadcrumbs).toHaveLength(1); - expect(breadcrumbs![0]?.data?.url).toEqual('http://example.com/pass'); + expect(breadcrumbs![0]?.data?.url).toEqual('https://example.com/pass'); }, }) .start(done); @@ -157,11 +157,11 @@ describe('httpIntegration', () => { const requestSpans = event.spans?.filter(span => span.op === 'http.client'); expect(requestSpans).toHaveLength(1); - expect(requestSpans![0]?.description).toBe('GET http://example.com/pass'); + expect(requestSpans![0]?.description).toBe('GET https://example.com/pass'); const breadcrumbs = event.breadcrumbs?.filter(b => b.category === 'http'); expect(breadcrumbs).toHaveLength(1); - expect(breadcrumbs![0]?.data?.url).toEqual('http://example.com/pass'); + expect(breadcrumbs![0]?.data?.url).toEqual('https://example.com/pass'); }, }) .start(done);