[PR #567] [MERGED] Do not send ACL unless overridden #1552

New issue

Closed

opened 2026-03-04 02:01:01 +03:00 by kerem · 0 comments

kerem commented

2026-03-04 02:01:01 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/s3fs-fuse/s3fs-fuse/pull/567
Author: @gaul
Created: 4/22/2017
Status: ✅ Merged
Merged: 5/9/2017
Merged by: @ggtakec

Base: master ← Head: default-acl

📝 Commits (1)

7b30d5d Do not send canned ACL header when empty string

📊 Changes

2 files changed (+12 additions, -5 deletions)

View changed files

📝 src/curl.cpp (+9 -3)
📝 src/s3fs_util.cpp (+3 -2)

📄 Description

Some providers such as StorageGRID do not support canned ACLs. Amazon
defaults to the private canned ACL if not provided so do not send it
unless overridden. References #125.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/s3fs-fuse/s3fs-fuse/pull/567 **Author:** [@gaul](https://github.com/gaul) **Created:** 4/22/2017 **Status:** ✅ Merged **Merged:** 5/9/2017 **Merged by:** [@ggtakec](https://github.com/ggtakec) **Base:** `master` ← **Head:** `default-acl` --- ### 📝 Commits (1) - [`7b30d5d`](https://github.com/s3fs-fuse/s3fs-fuse/commit/7b30d5d15b6999b8b4fb3f221004c8a49c185c4e) Do not send canned ACL header when empty string ### 📊 Changes **2 files changed** (+12 additions, -5 deletions) <details> <summary>View changed files</summary> 📝 `src/curl.cpp` (+9 -3) 📝 `src/s3fs_util.cpp` (+3 -2) </details> ### 📄 Description Some providers such as StorageGRID do not support canned ACLs. Amazon defaults to the private canned ACL if not provided so do not send it unless overridden. References #125. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>