-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add support for --profile for selecting IAM credentials #131
Conversation
Why not set the This way no change to aws-sam-local is required. |
|
@@ -108,6 +108,10 @@ func main() { | |||
Usage: "Optional. Specify whether SAM should skip pulling down the latest Docker image. Default is false.", | |||
EnvVar: "SAM_SKIP_PULL_IMAGE", | |||
}, | |||
cli.StringFlag{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Notes for future: common options like profile must be abstracted into an args
object and passed down to Invoke/StartApi for processing.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for doing it!
Is this flag in the current version of sam-cli? It doesn't seem to show up in 0.15 from Brew:
|
@cam8001 It is but at the end of the command |
Related Issue: #27
Adds --profile as a new flag for start-api and invoke to allow user to specify specific profiles if needed, instead of always using the default profile