Skip to content
This repository was archived by the owner on Jun 5, 2024. It is now read-only.

Commit 77593d3

Browse files
committed
fix: added @smithy/smithy-client registration to nr-hooks.js because of https://github.com/aws/aws-sdk-js-v3/pull/4873/files.
1 parent 55ac220 commit 77593d3

File tree

4 files changed

+18
-3
lines changed

4 files changed

+18
-3
lines changed

nr-hooks.js

+6
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,12 @@ const instrumentations = [
1717
onResolved: require('./lib/v3/smithy-client'),
1818
shimName: 'aws-sdk'
1919
},
20+
{
21+
type: 'generic',
22+
moduleName: '@smithy/smithy-client',
23+
onResolved: require('./lib/v3/smithy-client'),
24+
shimName: 'aws-sdk'
25+
},
2026
{
2127
type: 'message',
2228
moduleName: '@aws-sdk/client-sns',

tests/versioned/v3/client-dynamodb.tap.js

+5-1
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,11 @@ tap.test('DynamoDB', (t) => {
9696
DynamoDBClient = null
9797

9898
Object.keys(require.cache).forEach((key) => {
99-
if (key.includes('@aws-sdk/client-dynamodb') || key.includes('@aws-sdk/smithy-client')) {
99+
if (
100+
key.includes('@aws-sdk/client-dynamodb') ||
101+
key.includes('@aws-sdk/smithy-client') ||
102+
key.includes('@smithy/smithy-client')
103+
) {
100104
delete require.cache[key]
101105
}
102106
})

tests/versioned/v3/lib-dynamodb.tap.js

+2-1
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,8 @@ tap.test('DynamoDB', (t) => {
7070
if (
7171
key.includes('@aws-sdk/lib-dynamodb') ||
7272
key.includes('@aws-sdk/client-dynamodb') ||
73-
key.includes('@aws-sdk/smithy-client')
73+
key.includes('@aws-sdk/smithy-client') ||
74+
key.includes('@smithy/smithy-client')
7475
) {
7576
delete require.cache[key]
7677
}

tests/versioned/v3/sns.tap.js

+5-1
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,11 @@ tap.test('SNS', (t) => {
4949
// which files within the modules were cached preventing the instrumenting
5050
// from running on every test
5151
Object.keys(require.cache).forEach((key) => {
52-
if (key.includes('@aws-sdk/client-sns') || key.includes('@aws-sdk/smithy-client')) {
52+
if (
53+
key.includes('@aws-sdk/client-sns') ||
54+
key.includes('@aws-sdk/smithy-client') ||
55+
key.includes('@smithy/smithy-client')
56+
) {
5357
delete require.cache[key]
5458
}
5559
})

0 commit comments

Comments
 (0)