Skip to content

fix: Remove presignedUrl ACL #441

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weโ€™ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
char-yb merged 2 commits into from
Sep 26, 2025
Merged

fix: Remove presignedUrl ACL #441

char-yb merged 2 commits into from
Sep 26, 2025

Conversation

@char-yb
Copy link
Member

@char-yb char-yb commented Sep 26, 2025 •
edited
Loading

๐ŸŒฑ ๊ด€๋ จ ์ด์Šˆ

๐Ÿ“Œ ์ž‘์—… ๋‚ด์šฉ ๋ฐ ํŠน์ด์‚ฌํ•ญ

  • R2 Storage๋Š” ACL์„ ์ง€์›ํ•˜์ง€ ์•Š๊ฑฐ๋‚˜ ๋ฌด์‹œํ•˜๋Š” ๊ตฌ์„ฑ์ด ํ”ํ•จ. x-amz-acl์„ ์„œ๋ช…์— ๋„ฃ์œผ๋ฉด ํ˜ธํ™˜์„ฑ ์ด์Šˆ๋กœ 403์ด ๋‚  ์ˆ˜ ์žˆ์œผ๋ฏ€๋กœ acl ์‚ฌํ•ญ ์ œ๊ฑฐ

๐Ÿ“ ์ฐธ๊ณ ์‚ฌํ•ญ

๐Ÿ“š ๊ธฐํƒ€

@char-yb char-yb self-assigned this Sep 26, 2025
@char-yb char-yb linked an issue Sep 26, 2025 that may be closed by this pull request
๐Ÿ› PresignedUrl ACL ์‚ฌํ•ญ ์ œ๊ฑฐ #440
Closed
@10mm-automation-bot 10mm-automation-bot bot moved this to ์ฒ˜๋ฆฌ ๋Œ€๊ธฐ in 10mm-server Sep 26, 2025
@char-yb char-yb added ๐Ÿฅ‡ P1 ๊ธ‰ํ•˜๊ณ  ๊ผญ ํ•„์š”ํ•œ ์ด์Šˆ ๐Ÿ› bug/error ๋ฒ„๊ทธ ๋ฐ ์—๋Ÿฌ ํ”ฝ์Šค labels Sep 26, 2025
uwoobeat
uwoobeat approved these changes Sep 26, 2025
View reviewed changes
Copy link
Member

@uwoobeat uwoobeat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@char-yb char-yb merged commit 518f004 into develop Sep 26, 2025
1 check passed
@github-project-automation github-project-automation bot moved this from ์ฒ˜๋ฆฌ ๋Œ€๊ธฐ to ์™„๋ฃŒ in 10mm-server Sep 26, 2025
@char-yb char-yb deleted the fix/440-remove-presigned-url-acl branch September 26, 2025 12:08
@github-actions github-actions bot added the merged ๋จธ์ง€๋œ PR label Sep 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@uwoobeat uwoobeat uwoobeat approved these changes

@kdomo kdomo Awaiting requested review from kdomo kdomo is a code owner

Assignees

@char-yb char-yb

Labels

๐Ÿ› bug/error ๋ฒ„๊ทธ ๋ฐ ์—๋Ÿฌ ํ”ฝ์Šค merged ๋จธ์ง€๋œ PR ๐Ÿฅ‡ P1 ๊ธ‰ํ•˜๊ณ  ๊ผญ ํ•„์š”ํ•œ ์ด์Šˆ

Projects

10mm-server
Status: ์™„๋ฃŒ

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

๐Ÿ› PresignedUrl ACL ์‚ฌํ•ญ ์ œ๊ฑฐ

3 participants

@char-yb @uwoobeat