apache / apache/opendal

new feature: Add support for `PutObjectAcl` on objects.

Open
#7,973 4 comments 1 reaction 0 assignees View on GitHub
enhancement releases-note/feat services/s3
Dominant language
Rust
Stars
5.4k
Forks
825
Avg merge
1d 14m
Merged PRs (30d)
127

Description

### Feature Description

I want to change existing object's ACL.

(And override the default ACL if we can).

### Problem and Solution

#5358 says we need a way to set the ACL for objects and commit [740795c](https://github.com/antoine-de/opendal/commit/740795c9a200f8924ee5cb2f451e0f5f76e3bbb3) do add the default ACL... This commit make we can write objects with the default ACL but we cannot:

- Change existing object's ACL.
- Or override the default ACL (e.g. the default ACL is `public-read` but for object `foobar.secret`, I want to make it `private-read`).

### Additional Context

_No response_

### Are you willing to contribute to the development of this feature?

- [x] Yes, I am willing to contribute to the development of this feature.

Contributor guide

Open the contributing guide

Research direction

Start by reading commit 740795c and tracing the existing default-ACL support it introduced. Then locate the object-writing and ACL-related entry points and determine how changing an existing object's ACL and overriding the default ACL should be represented. Done means both requested operations are supported and covered by the relevant tests.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
backend-api-design
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.