Skip to content

Conversation

@sgore-godaddy
Copy link
Contributor

Summary

This change removes the object-level ACL parameter from S3 putObject calls, eliminating the need for the s3:PutObjectAcl IAM permission.

Motivation

Object-level ACLs are being phased out in favor of bucket policies as an AWS best practice. By removing the ACL: 'public-read' parameter, we can tighten IAM permissions and manage public access centrally through bucket policy instead.

Changelog

  • Removed ACL parameter from uploadFileToStorage function
  • Removed unused DEFAULT_ACL constant
  • Public read access is now controlled via bucket policy rather than per-object ACLs

Test Plan

@sgore-godaddy sgore-godaddy merged commit 767ea86 into main Oct 30, 2025
1 check passed
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