Skip to content

fix: prevent content deletion from artifact bucket#1073

Merged
ayush-panta merged 2 commits intomainfrom
add-guardrails-s3-bucket
Jan 29, 2026
Merged

fix: prevent content deletion from artifact bucket#1073
ayush-panta merged 2 commits intomainfrom
add-guardrails-s3-bucket

Conversation

@coderbirju
Copy link
Contributor

@coderbirju coderbirju commented Jan 15, 2026

Issue #, if available:

  • prevent deployments from deleting s3 bucket contents
  • add s3 listObject permissions to allow cloudfrontOAI find the bucket

Description of changes:

Testing done:

  • I've reviewed the guidance in CONTRIBUTING.md

License Acceptance

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Signed-off-by: Arjun Raja Yogidas <arjunry@amazon.com>
@coderbirju coderbirju requested a review from a team as a code owner January 15, 2026 23:56
pendo324
pendo324 previously approved these changes Jan 16, 2026
@coderbirju coderbirju dismissed stale reviews from Swapnanil-Gupta and pendo324 via b69a699 January 16, 2026 20:04
@coderbirju coderbirju force-pushed the add-guardrails-s3-bucket branch 2 times, most recently from 3de8a4f to 3c46ab8 Compare January 16, 2026 20:11
Signed-off-by: Arjun Raja Yogidas <arjunry@amazon.com>
@coderbirju coderbirju force-pushed the add-guardrails-s3-bucket branch from 3c46ab8 to 85884a5 Compare January 16, 2026 20:57
@ayush-panta ayush-panta merged commit 94d09a7 into main Jan 29, 2026
6 checks passed
@ayush-panta ayush-panta deleted the add-guardrails-s3-bucket branch January 29, 2026 21:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants