Skip to content
This repository has been archived by the owner on Feb 3, 2021. It is now read-only.

feat(aws): Support aws-vault as an option instead of named AWS profiles #206

Merged
merged 1 commit into from
Aug 21, 2019

Conversation

kouk
Copy link
Contributor

@kouk kouk commented Aug 21, 2019

Description

Update the AWS function to optionally retrieve the profile name from the AWS_VAULT environment variable instead of AWS_PROFILE.

Motivation and Context

When using 99designs/aws-vault the variable AWS_VAULT is set to indicate which profile has been configured in the environment and the AWS_PROFILE environment variable is not set. This commits makes AWS_VAULT override the profile name.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

How Has This Been Tested?

  • I have tested using MacOS
  • I have tested using Linux

Checklist:

  • I have checked that no other PR duplicates mine
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have updated the tests accordingly.

When using [99designs/aws-vault](https://github.com/99designs/aws-vault) the variable `AWS_VAULT` is set to indicate which profile has been configured in the environment and the AWS_PROFILE environment variable is not set. This commits makes AWS_VAULT override the profile name.
Copy link
Owner

@matchai matchai left a comment

Choose a reason for hiding this comment

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

Flawless PR ✨
Thank you for your contribution! 😄

@matchai matchai merged commit 5e02e3f into matchai:master Aug 21, 2019
@matchai
Copy link
Owner

matchai commented Aug 21, 2019

@all-contributors please add @kouk for code and test.

@allcontributors
Copy link
Contributor

@matchai

I've put up a pull request to add @kouk! 🎉

matchai pushed a commit that referenced this pull request Aug 21, 2019
# [2.7.0](v2.6.1...v2.7.0) (2019-08-21)

### Features

* **aws:** Support aws-vault as an option instead of named AWS p… ([#206](#206)) ([5e02e3f](5e02e3f))
@matchai
Copy link
Owner

matchai commented Aug 21, 2019

🎉 This PR is included in version 2.7.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants