tracker issue : CF-4202339

select a category, or use search below
(searches all categories and all time range)
Title:

cffile action="rename" causes S3 file to lose it's previous ACL value(s)

| View in Tracker

Status/Resolution/Reason: Closed/Fixed/Fixed

Reporter/Name(from Bugbase): Denard Springle / ()

Created: 05/08/2018

Components: File Management, VFS-S3

Versions: 2016

Failure Type: Data Loss

Found In Build/Fixed In Build: Latest / 2020.0.0.314573

Priority/Frequency: Normal / All users will encounter

Locale/System: ALL / Win 2016

Vote Count: 0

Problem Description:

After using fileMove() to copy a file to Amazon S3 and using setStoreACL() to add an ACL to it, then using cffile action="rename" to rename that file, it looses the previously set ACL. 

Steps to Reproduce:

1. Move a file to S3 with fileMove()
2. add an ACL to that file with setStoreACL()
3. rename the file with cffile action="rename"

Actual Result:

File looses the previously stored ACL information and reverts to bucket ACL

Expected Result:

File retains the previously stored ACL information

Any Workarounds:

Rename the file before calling storeAddACL() (or run storeAddACL() again after the rename - which is the current workaround in this clunky old spaghetti code I'm working on atm)

Attachments:

Comments:

I have not tried, but assume the same conclusion will happen with fileRename().
Comment by Denard S.
27718 | May 08, 2018 07:30:59 PM GMT
Hi Denard, Just FYI, a related issue that has seemingly been fixed at least twice is: CF-3043124,CF-3863487 Apparently its a safer bet to call StoreSetMetadata() before calling StoreAddACL(). Not sure if that'd pertain to anything ur doing, but just mentioning it as an FYI. And thanks very much for filling this :) I'm looking to implement a long-awaited S3 integration before year-end. Possibly. We'll see :) So, of course I hope Adobe gets S3 tightened-up. Thanks!, -Aaron
Comment by Aaron N.
27776 | May 13, 2018 07:46:37 AM GMT
I guess it's a common theme that CF's S3 integration is finicky when it comes to ACLs. So I guess we should tread cautiously ;) Thanks!, -Aaron
Comment by Aaron N.
27777 | May 13, 2018 07:48:24 AM GMT