Unverified Commit 96c1cd1f authored by Tom Forbes's avatar Tom Forbes Committed by GitHub
Browse files

Switch to debug logging for credential fetches (#3546) 

By submitting this pull request, I confirm that you can use, modify,
copy, and redistribute this contribution, under the terms of your
choice.
parent 67704a27

CHANGELOG.next.toml

+6 −0
Original line number Diff line number Diff line
@@ -28,3 +28,9 @@ message = "Make `SigningSettings` and its fields implement `Clone` and `Copy`" 
references = ["smithy-rs#3533"]

meta = { "breaking" = false, "tada" = false, "bug" = false }

author = "avandesa"



[[aws-sdk-rust]]

message = "Change some credentials related info log messages to debug."

references = ["smithy-rs#3546"]

meta = { "breaking" = false, "tada" = false, "bug" = false }

author = "orf"
 
 No newline at end of file

aws/rust-runtime/aws-config/src/profile/credentials/exec.rs

+2 −2
Original line number Diff line number Diff line
@@ -160,12 +160,12 @@ impl ProviderChain { 
                }

            }

        };

        tracing::info!(base = ?repr.base(), "first credentials will be loaded from {:?}", repr.base());

        tracing::debug!(base = ?repr.base(), "first credentials will be loaded from {:?}", repr.base());

        let chain = repr

            .chain()

            .iter()

            .map(|role_arn| {

                tracing::info!(role_arn = ?role_arn, "which will be used to assume a role");

                tracing::debug!(role_arn = ?role_arn, "which will be used to assume a role");

                AssumeRoleProvider {

                    role_arn: role_arn.role_arn.into(),

                    external_id: role_arn.external_id.map(Into::into),

rust-runtime/aws-smithy-runtime/src/client/identity/cache/lazy.rs

+1 −1
Original line number Diff line number Diff line
@@ -354,7 +354,7 @@ impl ResolveCachedIdentity for LazyCache { 
                            // `cache.get_or_load`, logging inside `cache.get_or_load` ensures that it is emitted

                            // only once for the first thread that succeeds in populating a cache value.

                            let printable = DateTime::from(expiration);

                            tracing::info!(

                            tracing::debug!(

                                new_expiration=%printable,

                                valid_for=?expiration.duration_since(time_source.now()).unwrap_or_default(),

                                partition=?partition,