Closed rtyler closed 1 week ago
Attention: Patch coverage is 84.14634%
with 13 lines
in your changes missing coverage. Please review.
Project coverage is 72.33%. Comparing base (
b4eff70
) to head (3a25235
). Report is 1 commits behind head on main.
Files with missing lines | Patch % | Lines |
---|---|---|
crates/aws/src/credentials.rs | 84.14% | 0 Missing and 13 partials :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
object_store
invokesget_credential
on every invocation of a get/list/put/etc. The provider invocation for environment based credentials is practically zero-cost, so this has no/low overhead.In the case of the AssumeRoleProvider or any provider which has some cost, such as an invocation of the AWS STS APIs, this can result in rate-limiting or service quota exhaustion.
In order to prevent this, the credentials are attempted to be cached only so long as they have no expired, which is defined in the
aws_credential_types::Credential
structSponsored-by: Scribd Inc