Skip to content

Commit

Permalink
chore(region-info): cn-northwest-1 (Ningxia) ROUTE_53_BUCKET_WEBSITE_…
Browse files Browse the repository at this point in the history
…ZONE_ID (aws#17024)


https://docs.aws.amazon.com/general/latest/gr/s3.html#s3_region, hashicorp/terraform-provider-aws#21337


----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
  • Loading branch information
PatMyron authored and TikiTDO committed Feb 21, 2022
1 parent 89f6f4b commit c7876ea
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
Expand Up @@ -74,10 +74,10 @@ test('throws if region agnostic', () => {
}).toThrow(/Cannot use an S3 record alias in region-agnostic stacks/);
});

test('throws if bucket website hosting is unavailable (cn-northwest-1)', () => {
test('throws if bucket website hosting is unavailable (cn-north-1)', () => {
// GIVEN
const app = new App();
const stack = new Stack(app, 'test', { env: { region: 'cn-northwest-1' } });
const stack = new Stack(app, 'test', { env: { region: 'cn-north-1' } });

const bucketWebsite = new s3.Bucket(stack, 'Bucket');

Expand Down
3 changes: 2 additions & 1 deletion packages/@aws-cdk/region-info/build-tools/fact-tables.ts
Expand Up @@ -43,7 +43,7 @@ export const AWS_CDK_METADATA = new Set([
/**
* The hosted zone Id if using an alias record in Route53.
*
* @see https://docs.aws.amazon.com/general/latest/gr/rande.html#s3_website_region_endpoints
* @see https://docs.aws.amazon.com/general/latest/gr/s3.html#s3_region
*/
export const ROUTE_53_BUCKET_WEBSITE_ZONE_IDS: { [region: string]: string } = {
'af-south-1': 'Z11KHD8FBVPUYU',
Expand All @@ -55,6 +55,7 @@ export const ROUTE_53_BUCKET_WEBSITE_ZONE_IDS: { [region: string]: string } = {
'ap-southeast-1': 'Z3O0J2DXBE1FTB',
'ap-southeast-2': 'Z1WCIGYICN2BYD',
'ca-central-1': 'Z1QDHH18159H29',
'cn-northwest-1': 'Z282HJ1KT0DH03',
'eu-central-1': 'Z21DNDUVLTQW6Q',
'eu-north-1': 'Z3BAZG2TWCNX0D',
'eu-south-1': 'Z3IXVV8C73GIO3',
Expand Down

0 comments on commit c7876ea

Please sign in to comment.