ondemand icon indicating copy to clipboard operation
ondemand copied to clipboard

show error when chown fails on upload

Open johrstrom opened this issue 1 year ago • 1 comments

#3856 is about to fix a bug when the chown operation fails in the files controller.

Instead of silently failing - it should actually just inform the user that the file did indeed upload, but it cannot chown the file. However, trying to get that implemented right now with Uppy is a little bit more than I can do (I want to get it backported to 3.1 ASAP).

So while we're stuck with #3856 for however long, we should actually refactor this failure mode (and possibly alerting) so that the user is informed about what's happened.

johrstrom avatar Oct 07 '24 18:10 johrstrom

Thanks for taking my feedback and questions into consideration!

moffatsadeghi avatar Oct 07 '24 19:10 moffatsadeghi

Review notes:

Explained that when file ownership changes fail during uploads, it fails silently Suggested it seems straightforward to implement since there's already a rescue block Noted version 4.1 is already quite full with tasks

Scheduled for version 4.2

moffatsadeghi avatar Sep 22 '25 15:09 moffatsadeghi