-
-
Notifications
You must be signed in to change notification settings - Fork 93
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
better support for stages #28
Conversation
scripts/deploy.sh
Outdated
@@ -20,5 +20,5 @@ source ./config.sh | |||
|
|||
OUTPUT_TEMPLATE_FILE="$PROJECT_ROOT/serverless-output.yml" | |||
aws s3 mb "s3://$BUCKET_NAME" --region "$REGION" || true | |||
sam deploy --region "$REGION" --template-file "$OUTPUT_TEMPLATE_FILE" --stack-name "$STACK_NAME" --parameter-overrides GitHubClientIdParameter="$GITHUB_CLIENT_ID" GitHubClientSecretParameter="$GITHUB_CLIENT_SECRET" CognitoRedirectUriParameter="$COGNITO_REDIRECT_URI" StageNameParameter="$STAGE_NAME" --capabilities CAPABILITY_IAM |
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.
pass in the stage
Type: AWS::Serverless::Api | ||
Properties: | ||
StageName: !Ref StageNameParameter | ||
OpenApiVersion: "2.0" |
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.
needed to get around the sam
stage bug: aws/serverless-application-model#191
Thanks very much for this! I especially appreciate the documentation updates 🙏 |
happy to help @TimothyJones! |
@TimothyJones not sure what the process is here, but this pr introduces an intentional api gateway that allows us to target custom stages and not generate a random stage named
Stage
. these changes are deployed in a development environment and tested against that with success (minus github 2fa users as discussed in #24).