-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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(route53): remove http://
from bucket target
#4070
Conversation
* change bucket type to IBucket to allow static import
Thanks so much for taking the time to contribute to the AWS CDK ❤️ We will shortly assign someone to review this pull request and help get it
|
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
http://
from bucket targethttp://
from bucket target
Are you sure the regional domain same has the same semantic as the website url with regards to website hosting? |
Good catch @hoegertn, you're right, it doesn't include I'll look into it truncating the |
I am currently working on a PR for r53 redirects. Within this pr I added a websiteDomainName attribute to the bucket. Maybe this would help once it is merged... |
Yeah, |
@eladb Should be good for review 👍 |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
I am not sure about the second test. SHouldn't the Join contain |
You are right this is a bug in the Bucket, and I just used the existing |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
Thank you for contributing! Your pull request is now being automatically merged. |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
* fix(route53-targets): remove `http://` from bucket target * change bucket type to IBucket to allow static import * chore: tslint fix * fix: use bucketWebsiteDomainName property * chore: remove unnecessary quotes
Fixes #3928
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license