Skip to content

Skip empty configuration values when writing AWS CLI config#10003

Open
Gopesh111 wants to merge 3 commits intoaws:developfrom
Gopesh111:develop
Open

Skip empty configuration values when writing AWS CLI config#10003
Gopesh111 wants to merge 3 commits intoaws:developfrom
Gopesh111:develop

Conversation

@Gopesh111
Copy link
Copy Markdown

Issue #, if available:
N/A

Description of changes:
This change prevents empty or whitespace-only configuration values from being
written to AWS CLI config files.

Previously, passing an empty value could result in invalid or confusing
configuration entries (e.g., region =), which may cause unexpected behavior
later. This update skips writing such values while preserving existing behavior
for valid inputs.

A unit test is included to cover this edge case.

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

@Gopesh111
Copy link
Copy Markdown
Author

Hi team, I just wanted to follow up on this PR to see if there’s anything else needed from my side. I’ve included unit tests to ensure empty configuration values are skipped as described. Happy to make any adjustments based on your feedback!

@Gopesh111
Copy link
Copy Markdown
Author

Hi @aws-cli-maintainers,

I'm following up on this PR to see if it can be scheduled for review. This change addresses a small but important edge case where empty or whitespace-only values are written to the AWS CLI config, potentially causing issues with downstream commands.

I have already included unit tests covering this behavior. Please let me know if there are any specific AWS contribution guidelines or further tests you'd like me to implement to move this forward.

Thank you!

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.

2 participants