{"action":"created","issue":{"url":"https://api.github.com/repos/ncw/rclone/issues/631","repository_url":"https://api.github.com/repos/ncw/rclone","labels_url":"https://api.github.com/repos/ncw/rclone/issues/631/labels{/name}","comments_url":"https://api.github.com/repos/ncw/rclone/issues/631/comments","events_url":"https://api.github.com/repos/ncw/rclone/issues/631/events","html_url":"https://github.com/ncw/rclone/pull/631","id":172450849,"number":631,"title":"User-configurable Amazon S3 ACL","user":{"login":"radek-senfeld","id":1120464,"avatar_url":"https://avatars.githubusercontent.com/u/1120464?v=3","gravatar_id":"","url":"https://api.github.com/users/radek-senfeld","html_url":"https://github.com/radek-senfeld","followers_url":"https://api.github.com/users/radek-senfeld/followers","following_url":"https://api.github.com/users/radek-senfeld/following{/other_user}","gists_url":"https://api.github.com/users/radek-senfeld/gists{/gist_id}","starred_url":"https://api.github.com/users/radek-senfeld/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/radek-senfeld/subscriptions","organizations_url":"https://api.github.com/users/radek-senfeld/orgs","repos_url":"https://api.github.com/users/radek-senfeld/repos","events_url":"https://api.github.com/users/radek-senfeld/events{/privacy}","received_events_url":"https://api.github.com/users/radek-senfeld/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-08-22T13:21:51Z","updated_at":"2016-08-22T14:42:03Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/ncw/rclone/pulls/631","html_url":"https://github.com/ncw/rclone/pull/631","diff_url":"https://github.com/ncw/rclone/pull/631.diff","patch_url":"https://github.com/ncw/rclone/pull/631.patch"},"body":"Hi,\r\nI needed to apply \"public-read\" ACL during upload to our Ceph/RadosGw. This simple patch works for me for now (newly uploaded objects have the right ACL, but subsequent sync doesn't seem to check/fix existing ones).\r\n\r\nWhat was your idea to fix the FIXME? ;-) I would love to see official support for S3 _canned_ ACL in rclone and might spend some time implementing it."},"comment":{"url":"https://api.github.com/repos/ncw/rclone/issues/comments/241434958","html_url":"https://github.com/ncw/rclone/pull/631#issuecomment-241434958","issue_url":"https://api.github.com/repos/ncw/rclone/issues/631","id":241434958,"user":{"login":"ncw","id":536803,"avatar_url":"https://avatars.githubusercontent.com/u/536803?v=3","gravatar_id":"","url":"https://api.github.com/users/ncw","html_url":"https://github.com/ncw","followers_url":"https://api.github.com/users/ncw/followers","following_url":"https://api.github.com/users/ncw/following{/other_user}","gists_url":"https://api.github.com/users/ncw/gists{/gist_id}","starred_url":"https://api.github.com/users/ncw/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ncw/subscriptions","organizations_url":"https://api.github.com/users/ncw/orgs","repos_url":"https://api.github.com/users/ncw/repos","events_url":"https://api.github.com/users/ncw/events{/privacy}","received_events_url":"https://api.github.com/users/ncw/received_events","type":"User","site_admin":false},"created_at":"2016-08-22T14:42:03Z","updated_at":"2016-08-22T14:42:03Z","body":"Thanks for looking at this.\r\n\r\nTo finish the feature you would need to\r\n  * add a config section - find out the possible values for the acl and let the user choose them in `rlcone config`\r\n  * update the docs `docs/content/s3.md` - update the config walk through and add a new section about acl\r\n  * I think `perm` should be renamed `acl`\r\n  * Search for issues which you could close by implementing this (there is at least one) and mark your commit with `fixes #xxx`\r\n\r\n>  but subsequent sync doesn't seem to check/fix existing ones\r\n\r\nrclone doesn't sync any metadata except the modification time so I would think this is probably out of scope for rclone. The closest thing might be to make sure `SetModTime` updates the acl properly, then you tell the user to touch any files which need their permissions changing and do a sync."}}
{"actor":{"display_login":"ncw"}}