search

aws-quickstart / quickstart-microsoft-activedirectory

AWS Quick Start Team
Apache License 2.0
113 stars 113 forks source link

Flip condition #151

Closed fortran01 closed 2 years ago

fortran01 commented 2 years ago

Description of changes:

False needs to be true. :)

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

davmayd commented 2 years ago

That's incorrect. There are Regional Quick Start buckets (aws-quickstart-us-east-1, aws-quickstart-us-east-2, etc.), and we want to ensure that the assets are deployed from the correct ones. Thus, if the user keeps the default value for the bucket name ("aws-quickstart"), we want to append the Region to it.