Skip to content

feat: add auto credential option for cloud backends#347

Draft
thweetkomputer wants to merge 1 commit intomainfrom
feature/cloud-auto-credentials
Draft

feat: add auto credential option for cloud backends#347
thweetkomputer wants to merge 1 commit intomainfrom
feature/cloud-auto-credentials

Conversation

@thweetkomputer
Copy link
Collaborator

@thweetkomputer thweetkomputer commented Feb 5, 2026

Auto Credentials Support

  • Added cloud_auto_credentials flag to KvOptions, INI parsing, equality, and C/Rust bindings; new APIs let configs skip static cloud_access_key/cloud_secret_key.
  • AWS/GCS backend now selects DefaultAWSCredentialsProviderChain when the flag is enabled (with TODO for GCP-native sources) while tests mimic the same behavior; otherwise we
    keep the existing simple provider path.
  • Tests gain a provider member to hold the shared credential chain; defaults stay unchanged, so no IMDS call occurs unless the flag is enabled.
  • Work lives on branch feature/cloud-auto-credentials in commit Add auto credential option for cloud backends.

@coderabbitai
Copy link

coderabbitai bot commented Feb 5, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

  • 🔍 Trigger a full review
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feature/cloud-auto-credentials

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

bool enable)
{
if (opts)
reinterpret_cast<KvOptions *>(opts)->cloud_auto_credentials = enable;
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[cpplint] reported by reviewdog 🐶
Lines should be <= 80 characters long [whitespace/line_length] [2]

{
if (options_ && options_->cloud_auto_credentials)
{
// TODO: extend this to support GCP-native credential sources when
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[cpplint] reported by reviewdog 🐶
Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]

@thweetkomputer thweetkomputer changed the title Add auto credential option for cloud backends feat: add auto credential option for cloud backends Feb 5, 2026
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.

[Feature]: Support GCS credential discovery for cloud_auto_credentials

1 participant