Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

QSS3KeyPrefix should end with a '/' #50

Merged
merged 1 commit into from
Nov 6, 2018

Conversation

hierynomus
Copy link

If it doesn't, the bastion host cannot find its init script and subsequently the stack cannot boot.

If it doesn't, the bastion host cannot find its init script and subsequently the stack cannot boot.
Copy link
Member

@andrew-glenn andrew-glenn left a comment

Choose a reason for hiding this comment

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

I ran this through a regex checker, just to be sure. The proposed regex pattern behaves as intended. Thanks for this!

@andrew-glenn andrew-glenn merged commit d77be53 into aws-quickstart:develop Nov 6, 2018
hierynomus pushed a commit to hierynomus/quickstart-linux-bastion that referenced this pull request Nov 6, 2018
Sorry, In aws-quickstart#50 I forgot to also change the regex for the `QSS3KeyPrefix` in the master template.
tlindsay42 pushed a commit that referenced this pull request Jul 25, 2022
QSS3KeyPrefix should end with a '/'
tlindsay42 pushed a commit that referenced this pull request Jul 25, 2022
Sorry, In #50 I forgot to also change the regex for the `QSS3KeyPrefix` in the master template.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants