Skip to content

Commit c45206b

Browse files
authored
chore: always append PTEnv to UA string (#4522)
1 parent 85ae53c commit c45206b

File tree

2 files changed

+6
-3
lines changed

2 files changed

+6
-3
lines changed

packages/commons/src/index.ts

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,10 @@
11
import { PT_VERSION } from './version.js';
22

3+
const env = process.env.AWS_EXECUTION_ENV || 'NA';
34
if (!process.env.AWS_SDK_UA_APP_ID) {
4-
process.env.AWS_SDK_UA_APP_ID = `PT/TEST/${PT_VERSION}`;
5+
process.env.AWS_SDK_UA_APP_ID = `PT/NO-OP/${PT_VERSION}/PTEnv/${env}`;
56
} else {
6-
process.env.AWS_SDK_UA_APP_ID = `${process.env.AWS_SDK_UA_APP_ID}/PT/TEST/${PT_VERSION}`;
7+
process.env.AWS_SDK_UA_APP_ID = `${process.env.AWS_SDK_UA_APP_ID}/PT/NO-OP/${PT_VERSION}/PTEnv/${env}`;
78
}
89

910
export { addUserAgentMiddleware, isSdkClient } from './awsSdkUtils.js';

packages/commons/tests/unit/awsSdkUtils.test.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,9 @@ describe('Helpers: awsSdk', () => {
8484

8585
it('concatenates the PT AWS_SDK_UA_APP_ID when one is already set', () => {
8686
// Assess
87-
expect(process.env.AWS_SDK_UA_APP_ID).toEqual(`test/PT/TEST/${version}`);
87+
expect(process.env.AWS_SDK_UA_APP_ID).toEqual(
88+
`test/PT/NO-OP/${version}/PTEnv/NA`
89+
);
8890
});
8991

9092
describe('Function: customUserAgentMiddleware', () => {

0 commit comments

Comments
 (0)