Skip to content

Commit adad689

Browse files
authored
ref(server-utils)!: Replace deprecated net.* span attributes on db and messaging spans (#23422)
1 parent f73bf21 commit adad689

44 files changed

Lines changed: 164 additions & 183 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

dev-packages/deno-integration-tests/suites/orchestrion-mongo/test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ Deno.test('mongodb instrumentation: orchestrion:mongodb:command channel produces
6060
assertEquals(mongoSpan!.data?.['db.namespace'], 'mydb');
6161
assertEquals(mongoSpan!.data?.['db.collection.name'], 'users');
6262
assertEquals(mongoSpan!.data?.['db.operation.name'], 'find');
63-
assertEquals(mongoSpan!.data?.['net.peer.name'], '127.0.0.1');
64-
assertEquals(mongoSpan!.data?.['net.peer.port'], 27017);
63+
assertEquals(mongoSpan!.data?.['server.address'], '127.0.0.1');
64+
assertEquals(mongoSpan!.data?.['server.port'], 27017);
6565
assertEquals(mongoSpan!.data?.['sentry.origin'], 'auto.db.mongo');
6666
});

dev-packages/deno-integration-tests/suites/orchestrion-mongoose/test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ Deno.test('mongoose instrumentation: orchestrion:mongoose:model_save channel pro
6161
assertEquals(mongooseSpan!.data?.['db.collection.name'], 'blogposts');
6262
assertEquals(mongooseSpan!.data?.['db.operation.name'], 'save');
6363
assertEquals(mongooseSpan!.data?.['db.user'], 'root');
64-
assertEquals(mongooseSpan!.data?.['net.peer.name'], '127.0.0.1');
65-
assertEquals(mongooseSpan!.data?.['net.peer.port'], 27017);
64+
assertEquals(mongooseSpan!.data?.['server.address'], '127.0.0.1');
65+
assertEquals(mongooseSpan!.data?.['server.port'], 27017);
6666
assertEquals(mongooseSpan!.data?.['sentry.origin'], 'auto.db.mongoose');
6767
});

dev-packages/deno-integration-tests/suites/orchestrion-mysql/test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -92,8 +92,8 @@ Deno.test('mysql instrumentation: orchestrion:mysql:query channel produces a nes
9292
assertEquals(mysqlSpan!.description, 'SELECT 1 AS solution');
9393
assertEquals(mysqlSpan!.data?.['db.system.name'], 'mysql');
9494
assertEquals(mysqlSpan!.data?.['db.query.text'], 'SELECT 1 AS solution');
95-
assertEquals(mysqlSpan!.data?.['net.peer.name'], '127.0.0.1');
96-
assertEquals(mysqlSpan!.data?.['net.peer.port'], 3306);
95+
assertEquals(mysqlSpan!.data?.['server.address'], '127.0.0.1');
96+
assertEquals(mysqlSpan!.data?.['server.port'], 3306);
9797
assertEquals(mysqlSpan!.data?.['db.user'], 'root');
9898
assertEquals(mysqlSpan!.data?.['sentry.origin'], 'auto.db.mysql');
9999
});

dev-packages/deno-integration-tests/suites/orchestrion-mysql2/test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ Deno.test('mysql2 instrumentation: orchestrion:mysql2:query channel produces a n
5555
assertEquals(mysqlSpan!.data?.['db.query.text'], 'SELECT 1 AS solution');
5656
assertEquals(mysqlSpan!.data?.['db.namespace'], 'mydb');
5757
assertEquals(mysqlSpan!.data?.['db.user'], 'root');
58-
assertEquals(mysqlSpan!.data?.['net.peer.name'], '127.0.0.1');
59-
assertEquals(mysqlSpan!.data?.['net.peer.port'], 3306);
58+
assertEquals(mysqlSpan!.data?.['server.address'], '127.0.0.1');
59+
assertEquals(mysqlSpan!.data?.['server.port'], 3306);
6060
assertEquals(mysqlSpan!.data?.['sentry.origin'], 'auto.db.mysql2');
6161
});

dev-packages/deno-integration-tests/suites/orchestrion-postgres/test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -92,8 +92,8 @@ Deno.test('pg instrumentation: orchestrion:pg:query channel produces a nested db
9292
assertEquals(pgSpan!.description, 'SELECT 1 AS solution');
9393
assertEquals(pgSpan!.data?.['db.system.name'], 'postgresql');
9494
assertEquals(pgSpan!.data?.['db.query.text'], 'SELECT 1 AS solution');
95-
assertEquals(pgSpan!.data?.['net.peer.name'], '127.0.0.1');
96-
assertEquals(pgSpan!.data?.['net.peer.port'], 5432);
95+
assertEquals(pgSpan!.data?.['server.address'], '127.0.0.1');
96+
assertEquals(pgSpan!.data?.['server.port'], 5432);
9797
assertEquals(pgSpan!.data?.['db.user'], 'root');
9898
assertEquals(pgSpan!.data?.['sentry.origin'], 'auto.db.postgres');
9999
});

dev-packages/deno-integration-tests/suites/orchestrion-tedious/test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ Deno.test('tedious instrumentation: orchestrion:tedious:execSql channel produces
5858
assertEquals(tediousSpan!.data?.['db.namespace'], 'mydb');
5959
assertEquals(tediousSpan!.data?.['db.user'], 'sa');
6060
assertEquals(tediousSpan!.data?.['db.query.text'], 'SELECT 1');
61-
assertEquals(tediousSpan!.data?.['net.peer.name'], '127.0.0.1');
62-
assertEquals(tediousSpan!.data?.['net.peer.port'], 1433);
61+
assertEquals(tediousSpan!.data?.['server.address'], '127.0.0.1');
62+
assertEquals(tediousSpan!.data?.['server.port'], 1433);
6363
assertEquals(tediousSpan!.data?.['sentry.origin'], 'auto.db.tedious');
6464
});

dev-packages/e2e-tests/test-applications/astro-6-cf-workers/tests/db.test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,8 +20,8 @@ test('a real mysql query emits a db span with orchestrion-channel attributes', a
2020
expect(firstQuery!.data?.['sentry.origin']).toBe('auto.db.mysql');
2121
expect(firstQuery!.data?.['db.system.name']).toBe('mysql');
2222
expect(firstQuery!.data?.['db.query.text']).toBe('SELECT 1 + 1 AS solution');
23-
expect(firstQuery!.data?.['net.peer.name']).toBe('127.0.0.1');
24-
expect(firstQuery!.data?.['net.peer.port']).toBe(3306);
23+
expect(firstQuery!.data?.['server.address']).toBe('127.0.0.1');
24+
expect(firstQuery!.data?.['server.port']).toBe(3306);
2525
expect(firstQuery!.data?.['db.user']).toBe('root');
2626
});
2727

dev-packages/e2e-tests/test-applications/astro-7/tests/db.test.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -63,8 +63,8 @@ test('Instruments mysql automatically', async ({ baseURL }) => {
6363
'db.query.text': 'SELECT 1 + 1 AS solution',
6464
'db.user': 'root',
6565
'db.connection_string': expect.any(String),
66-
'net.peer.name': expect.any(String),
67-
'net.peer.port': 3306,
66+
'server.address': expect.any(String),
67+
'server.port': 3306,
6868
}),
6969
}),
7070
);
@@ -79,8 +79,8 @@ test('Instruments mysql automatically', async ({ baseURL }) => {
7979
'db.query.text': 'SELECT NOW()',
8080
'db.user': 'root',
8181
'db.connection_string': expect.any(String),
82-
'net.peer.name': expect.any(String),
83-
'net.peer.port': 3306,
82+
'server.address': expect.any(String),
83+
'server.port': 3306,
8484
}),
8585
}),
8686
);

dev-packages/e2e-tests/test-applications/bun-mysql/tests/mysql.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ test('mysql queries emit a db span with orchestrion-channel attributes', async (
2626
expect(firstQuery!.data?.['sentry.origin']).toBe('auto.db.mysql');
2727
expect(firstQuery!.data?.['db.system.name']).toBe('mysql');
2828
expect(firstQuery!.data?.['db.query.text']).toBe('SELECT 1 + 1 AS solution');
29-
expect(firstQuery!.data?.['net.peer.port']).toBe(3306);
29+
expect(firstQuery!.data?.['server.port']).toBe(3306);
3030
expect(firstQuery!.data?.['db.user']).toBe('root');
3131
});
3232

dev-packages/e2e-tests/test-applications/cloudflare-orchestrion-mysql/tests/mysql.test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,8 +27,8 @@ test('a real mysql query emits a db span with orchestrion-channel attributes', a
2727
expect(firstQuery!.data?.['sentry.origin']).toBe('auto.db.mysql');
2828
expect(firstQuery!.data?.['db.system.name']).toBe('mysql');
2929
expect(firstQuery!.data?.['db.query.text']).toBe('SELECT 1 + 1 AS solution');
30-
expect(firstQuery!.data?.['net.peer.name']).toBe('127.0.0.1');
31-
expect(firstQuery!.data?.['net.peer.port']).toBe(3306);
30+
expect(firstQuery!.data?.['server.address']).toBe('127.0.0.1');
31+
expect(firstQuery!.data?.['server.port']).toBe(3306);
3232
expect(firstQuery!.data?.['db.user']).toBe('root');
3333
});
3434

0 commit comments

Comments
 (0)