Skip to content

feat(aws): Add support for sourcing credentials from external process#67

Merged
huseyinbabal merged 1 commit intohuseyinbabal:masterfrom
dzvon:push-kznkwnukmlqr
Jan 14, 2026
Merged

feat(aws): Add support for sourcing credentials from external process#67
huseyinbabal merged 1 commit intohuseyinbabal:masterfrom
dzvon:push-kznkwnukmlqr

Conversation

@dzvon
Copy link
Contributor

@dzvon dzvon commented Jan 13, 2026

Related Issue

Closes #34

Description

  • Implemented credential_process support in config files.
  • Added in-memory caching for process-sourced credentials.
  • Automatic refresh of temporary credentials based on the Expiration field.
  • Caching of long-term credentials (no Expiration) to minimize process calls.
  • Added unit test test_credential_process_success to verify the implementation.

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Refactoring (no functional changes)

Checklist

  • I have linked this PR to an existing issue
  • My code follows the project's code style
  • I have tested my changes locally
  • I have added/updated relevant documentation (if applicable)
  • My changes don't introduce new warnings

Additional Notes

External credentials are handled almost the same way as in the AWS CLI. See the documentation: https://docs.aws.amazon.com/cli/latest/userguide/cli-configure-sourcing-external.html

@dzvon dzvon requested a review from huseyinbabal as a code owner January 13, 2026 02:55
@huseyinbabal
Copy link
Owner

Hi @dzvon thank you for your contribution ❤️ Just wanted to inform you about a maintenance in ci pipeline, I will review your pr after that improvement, that means I will request a rebase soon 😄 It won't take long

- Implemented `credential_process` support in config files.
- Added in-memory caching for process-sourced credentials.
- Automatic refresh of temporary credentials based on the `Expiration` field.
- Caching of long-term credentials (no `Expiration`) to minimize process calls.
- Added unit test `test_credential_process_success` to verify the implementation.
@dzvon dzvon force-pushed the push-kznkwnukmlqr branch from f14e4ac to 310326c Compare January 13, 2026 13:02
@dzvon
Copy link
Contributor Author

dzvon commented Jan 13, 2026

@huseyinbabal Sure, I rebased my branch onto master.

Copy link
Owner

@huseyinbabal huseyinbabal left a comment

Choose a reason for hiding this comment

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

Works like a charm 🚀

@huseyinbabal huseyinbabal merged commit 3ae66f3 into huseyinbabal:master Jan 14, 2026
@dzvon dzvon deleted the push-kznkwnukmlqr branch January 15, 2026 01:32
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.

Support credential_process in authentication

2 participants