Skip to content

Commit 1f8e1bb

Browse files
committed
ADD: configurable deployment bucket resource name
1 parent 624d720 commit 1f8e1bb

File tree

4 files changed

+15
-8
lines changed

4 files changed

+15
-8
lines changed

src/cli/providers/cloudFormation/context/resources.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ import { createStack, setStackParameter, getStackName } from './stack'
99
import { getBucketReference } from './s3StorageDeployment'
1010

1111
export { s3DeploymentBucket, s3DeploymentBucketParameter, s3 } from './s3Storage'
12-
export { S3_DEPLOYMENT_BUCKET_RESOURCE_NAME } from './s3StorageDeployment'
12+
export { getDeploymentBucketResourceName } from './s3StorageDeployment'
1313
export { apiGateway } from './apiGateway'
1414
export { sns } from './sns'
1515
export { cloudWatchEvent } from './cloudWatchEvent'

src/cli/providers/cloudFormation/context/s3Storage.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ import { ExecuteStep, executor } from '../../../context'
44
import { collectMetadata } from '../../../utilities/collectMetadata'
55
import { setResource } from '../utils'
66
import { createStack, setStackParameter, getStackName } from './stack'
7-
import { S3_DEPLOYMENT_BUCKET_RESOURCE_NAME } from './s3StorageDeployment'
7+
import { getDeploymentBucketResourceName } from './s3StorageDeployment'
88

99
export const S3_STORAGE_STACK = 'S3Stack'
1010

@@ -17,7 +17,7 @@ export const s3DeploymentBucket = ExecuteStep.register('S3-Deployment-Bucket', a
1717
"Type": "AWS::S3::Bucket"
1818
}
1919

20-
const bucketResourceName = S3_DEPLOYMENT_BUCKET_RESOURCE_NAME
20+
const bucketResourceName = await getDeploymentBucketResourceName()
2121
const resourceName = setResource(context, bucketResourceName, s3BucketResource)
2222

2323
context.CloudFormationTemplate.Outputs[`${resourceName}Name`] = {
@@ -30,7 +30,7 @@ export const s3DeploymentBucket = ExecuteStep.register('S3-Deployment-Bucket', a
3030

3131

3232
export const s3DeploymentBucketParameter = ExecuteStep.register('S3-Deployment-Bucket-Parameter', async (context) => {
33-
const resourceName = S3_DEPLOYMENT_BUCKET_RESOURCE_NAME
33+
const resourceName = await getDeploymentBucketResourceName()
3434
await setStackParameter({
3535
...context,
3636
resourceName
Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,13 @@
1-
export const S3_DEPLOYMENT_BUCKET_RESOURCE_NAME = 'FunctionlyDeploymentBucket'
1+
import { projectConfig } from '../../../project/config'
2+
3+
const S3_DEPLOYMENT_BUCKET_RESOURCE_NAME = 'FunctionlyDeploymentBucket'
4+
5+
export const getDeploymentBucketResourceName = async () => {
6+
return projectConfig.awsBucketResourceName || S3_DEPLOYMENT_BUCKET_RESOURCE_NAME
7+
}
28

39
export const getBucketReference = async (context) => {
410
return context.__userAWSBucket ? context.awsBucket : {
5-
"Ref": S3_DEPLOYMENT_BUCKET_RESOURCE_NAME
11+
"Ref": await getDeploymentBucketResourceName()
612
}
713
}

src/cli/providers/cloudFormation/index.ts

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ import { executor } from '../../context'
99
import { cloudFormationInit, cloudFormationMerge } from './context/cloudFormationInit'
1010
import {
1111
tableResources, lambdaResources, roleResources, s3DeploymentBucket, s3DeploymentBucketParameter,
12-
apiGateway, sns, s3, cloudWatchEvent, initStacks, lambdaLogResources, S3_DEPLOYMENT_BUCKET_RESOURCE_NAME, tableSubscribers
12+
apiGateway, sns, s3, cloudWatchEvent, initStacks, lambdaLogResources, getDeploymentBucketResourceName, tableSubscribers
1313
} from './context/resources'
1414
import { uploadTemplate, persistCreateTemplate } from './context/uploadTemplate'
1515

@@ -38,7 +38,8 @@ export const cloudFormation = {
3838
}
3939
}
4040
if (!context.awsBucket) {
41-
const bucketData = await executor({ ...context, LogicalResourceId: S3_DEPLOYMENT_BUCKET_RESOURCE_NAME }, describeStackResouce)
41+
const logicalResourceId = await getDeploymentBucketResourceName()
42+
const bucketData = await executor({ ...context, LogicalResourceId: logicalResourceId }, describeStackResouce)
4243
context.awsBucket = bucketData.StackResourceDetail.PhysicalResourceId
4344
}
4445

0 commit comments

Comments
 (0)