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

Toolkit: toolkit output domain name needs to use RegionalDomainName for GovCloud and CN compatibility #1469

Closed
rix0rrr opened this issue Jan 2, 2019 · 5 comments · Fixed by #4427
Assignees
Labels
bug This issue is a bug. effort/small Small work item – less than a day of effort package/tools Related to AWS CDK Tools or CLI

Comments

@rix0rrr
Copy link
Contributor

rix0rrr commented Jan 2, 2019

Although { "Fn::GetAtt": "DomainName" } seems to work in CN, it does not work in GovCloud (in GovCloud the wrong domain name is returned).

RegionalDomainName seems to work; don't know if that will break something else.

@rix0rrr rix0rrr added bug This issue is a bug. package/tools Related to AWS CDK Tools or CLI labels Jan 2, 2019
@rix0rrr rix0rrr added the effort/small Small work item – less than a day of effort label Jan 8, 2019
@eladb
Copy link
Contributor

eladb commented Apr 2, 2019

@RomainMuller @rix0rrr what would you recommend we do to resolve this?

@rix0rrr
Copy link
Contributor Author

rix0rrr commented Apr 2, 2019

Update the bootstrap template and test it in a couple of commercial regions.

@beary
Copy link

beary commented Aug 2, 2019

I just tried in cn-north-1, { "Fn::GetAtt": "DomainName" } reports an error.

After modified the DomainName to RegionalDomainName, the problem above was resolved.

Value: { "Fn::GetAtt": ["StagingBucket", "DomainName"] }

@NGL321 NGL321 added response-requested Waiting on additional info and feedback. Will move to "closing-soon" in 7 days. and removed status/needs-response labels Sep 16, 2019
@whimzyLive
Copy link
Contributor

@rix0rrr @eladb This also doesn't work with CN regions.

@NGL321 NGL321 removed the response-requested Waiting on additional info and feedback. Will move to "closing-soon" in 7 days. label Oct 4, 2019
@sassoo
Copy link

sassoo commented Oct 4, 2019

specifically govcloud customers are being negatively impacted by this. cdk deploy's do not work at all.

@shivlaks shivlaks changed the title Toolkit: toolkit output domain name needs to use RegionalDomainName for GovCloud compatibility Toolkit: toolkit output domain name needs to use RegionalDomainName for GovCloud and CN compatibility Oct 7, 2019
shivlaks added a commit that referenced this issue Oct 9, 2019
@mergify mergify bot closed this as completed in #4427 Oct 10, 2019
mergify bot pushed a commit that referenced this issue Oct 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue is a bug. effort/small Small work item – less than a day of effort package/tools Related to AWS CDK Tools or CLI
Projects
None yet
7 participants