Skip to content

Commit b5ca7c4

Browse files
authored
fix(credential-provider-node): fix logger binding issue (#7302)
1 parent da65081 commit b5ca7c4

File tree

2 files changed

+6
-2
lines changed

2 files changed

+6
-2
lines changed

packages/credential-provider-http/src/fromHttp/fromHttp.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,9 @@ export const fromHttp = (options: FromHttpOptions = {}): AwsCredentialIdentityPr
2828
const tokenFile = options.awsContainerAuthorizationTokenFile ?? process.env[AWS_CONTAINER_AUTHORIZATION_TOKEN_FILE];
2929

3030
const warn: (warning: string) => void =
31-
options.logger?.constructor?.name === "NoOpLogger" || !options.logger ? console.warn : options.logger.warn;
31+
options.logger?.constructor?.name === "NoOpLogger" || !options.logger?.warn
32+
? console.warn
33+
: options.logger.warn.bind(options.logger);
3234

3335
if (relative && full) {
3436
warn(

packages/credential-provider-node/src/defaultProvider.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,9 @@ export const defaultProvider = (init: DefaultProviderInit = {}): MemoizedProvide
7070
if (envStaticCredentialsAreSet) {
7171
if (!multipleCredentialSourceWarningEmitted) {
7272
const warnFn =
73-
init.logger?.warn && init.logger?.constructor?.name !== "NoOpLogger" ? init.logger.warn : console.warn;
73+
init.logger?.warn && init.logger?.constructor?.name !== "NoOpLogger"
74+
? init.logger.warn.bind(init.logger)
75+
: console.warn;
7476
warnFn(
7577
`@aws-sdk/credential-provider-node - defaultProvider::fromEnv WARNING:
7678
Multiple credential sources detected:

0 commit comments

Comments
 (0)