adam/shoc-backend
1
0
Fork 0
mirror of https://github.com/Sea-Haven-Industries/shoc-backend.git synced 2026-09-30 07:13:12 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

fix: grant staging deploy role the Support-confirmed Elastic Beanstalk S3 set

Browse source 
s3:PutObject on shoc-backend/* covered only the upload the deploy action
performs itself. AWS Support case 178526484500047 established that
UpdateEnvironment then reads, writes, versions, ACL-checks and removes
objects as the calling identity, across both the account bucket and
AWS-owned Elastic Beanstalk service buckets whose names cannot be
enumerated in advance. That is the failure the dev lane already hit, and
staging calls the same deploy action and the same rollback
update-environment, so the first OIDC deploy and every rollback would
have stopped there.

Brings the staging role to parity with deploy-dev-stack.ts.
elasticbeanstalk:UpdateEnvironment stays pinned to the staging
environment ARN, so the role still cannot update or terminate
shoc-backend-dev.
This commit is contained in:
Alexandre Brandizzi 2026-08-31 10:57:04 -03:00
parent e51924eee6
commit 4aef5b463b
1 changed files with 31 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

43
infra/cdk/deploy-staging-stack.ts
View file

@ -114,25 +114,44 @@ export class DeployStagingStack extends cdk.Stack {
deployRole.addToPolicy( deployRole.addToPolicy(
new iam.PolicyStatement({ new iam.PolicyStatement({
effect: iam.Effect.ALLOW, effect: iam.Effect.ALLOW,
actions: ['s3:PutObject'], actions: ['s3:Delete*', 's3:Get*', 's3:Put*'],
// The pinned deployment action writes exactly // Matches deploy-dev-stack.ts. The narrower s3:PutObject on
// shoc-backend/<version-label>.zip to the explicitly configured, // shoc-backend/* covers only the upload the deploy action performs
// pre-existing account bucket. It never reads or deletes objects. // itself; AWS Support case 178526484500047 confirmed that
resources: [ // UpdateEnvironment then reads, writes, versions, ACL-checks, and
`arn:aws:s3:::elasticbeanstalk-${REGION}-${ACCOUNT_ID}/${APPLICATION_NAME}/*`, // removes objects as the calling identity, in both the account bucket
], // and AWS-owned Elastic Beanstalk service buckets whose names are not
// enumerable in advance. That is the failure the dev lane already hit;
// staging calls the same deploy action and the same rollback
// update-environment, so it needs the same set or the first OIDC
// deploy and every rollback stop there.
//
// NOTE: dev and staging are two environments of ONE Elastic Beanstalk
// application (APPLICATION_NAME is shared), so application versions for
// both live under the same shoc-backend/ prefix in the same account
// bucket. This grant therefore reaches dev's application-version
// objects. Environment authority stays separate --
// elasticbeanstalk:UpdateEnvironment below is pinned to the staging
// environment ARN, so this role still cannot update or terminate
// shoc-backend-dev.
resources: ['arn:aws:s3:::elasticbeanstalk-*/*'],
}), }),
); );
deployRole.addToPolicy( deployRole.addToPolicy(
new iam.PolicyStatement({ new iam.PolicyStatement({
effect: iam.Effect.ALLOW, effect: iam.Effect.ALLOW,
// HeadBucket on the explicit existing bucket requires ListBucket. actions: [
// GetBucketLocation is retained for regional SDK compatibility. 's3:GetBucket*',
actions: ['s3:GetBucketLocation', 's3:ListBucket'], 's3:ListBucket',
resources: [ 's3:PutBucketOwnershipControls',
`arn:aws:s3:::elasticbeanstalk-${REGION}-${ACCOUNT_ID}`, 's3:PutBucketPolicy',
's3:PutBucketPublicAccessBlock',
], ],
// AWS Support's bucket-level UpdateEnvironment set, excluding
// CreateBucket because the workflow deploys only to an existing
// application/environment and disables bucket creation.
resources: ['arn:aws:s3:::elasticbeanstalk-*'],
}), }),
); );