Skip to content
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

fix(buildspec): addons template url uses dot region #1695

Merged
merged 1 commit into from
Nov 17, 2020

Conversation

lukeledet
Copy link
Contributor

I saw this error in cloudformation when deploying a service via a pipeline: "TemplateURL must reference a valid S3 object to which you have access." Updating my buildspec with this change fixed the issue. S3 docs show this is a legacy URL that doesn't work for all regions.

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

@lukeledet lukeledet requested a review from a team as a code owner November 17, 2020 23:02
@efekarakus efekarakus changed the title Fix addons template url in buildspec fix(buildspec): addons template url in buildspec Nov 17, 2020
@efekarakus efekarakus changed the title fix(buildspec): addons template url in buildspec fix(buildspec): addons template url uses dot region Nov 17, 2020
Copy link
Contributor

@efekarakus efekarakus left a comment

Choose a reason for hiding this comment

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

Woohoo amazing! I had the same change locally :D but you got to it faster than me 🏃‍♂️

Copy link
Contributor

@iamhopaul123 iamhopaul123 left a comment

Choose a reason for hiding this comment

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

Thank you so much for fixing this! Could you update your commit with the conventional style?

For example fix: addons template url in buildspec

@mergify mergify bot merged commit 1714c8d into aws:mainline Nov 17, 2020
Sprint 🏃‍♀️ automation moved this from In review to Pending release Nov 17, 2020
@lukeledet lukeledet deleted the fix-addon-s3-url branch November 17, 2020 23:28
thrau pushed a commit to localstack/copilot-cli-local that referenced this pull request Dec 9, 2022
<!-- Provide summary of changes -->

I saw this error in cloudformation when deploying a service via a pipeline: `"TemplateURL must reference a valid S3 object to which you have access."` Updating my buildspec with this change fixed the issue. [S3 docs](https://docs.aws.amazon.com/AmazonS3/latest/dev/VirtualHosting.html) show this is a legacy URL that doesn't work for all regions.

<!-- Issue number, if available. E.g. "Fixes aws#31", "Addresses aws#42, 77" -->

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Sprint 🏃‍♀️
  
Pending release
Development

Successfully merging this pull request may close these issues.

None yet

3 participants