Skip to content

chore: align default credential provider API style#700

Open
Xuanwo wants to merge 1 commit intomainfrom
xuanwo/aws-style-audit
Open

chore: align default credential provider API style#700
Xuanwo wants to merge 1 commit intomainfrom
xuanwo/aws-style-audit

Conversation

@Xuanwo
Copy link
Member

@Xuanwo Xuanwo commented Feb 26, 2026

Summary

This PR aligns DefaultCredentialProvider / DefaultCredentialProviderBuilder API style across services using aws-v4 as the baseline.

Changes

  • Fix Oracle DefaultCredentialProvider::default() semantics to match new().
  • Add Oracle regression test to ensure default() and new() stay equivalent.
  • Export DefaultCredentialProviderBuilder from Volcengine provide_credential module.
  • Export DefaultCredentialProviderBuilder from Google and Oracle crate roots.
  • Make provide_credential module private in Oracle and Tencent (keep public re-exports).

Validation

  • cargo check -p reqsign-oracle -p reqsign-volcengine-tos -p reqsign-google -p reqsign-tencent-cos
  • cargo test -p reqsign-oracle test_default_matches_new

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant