gwu-libraries / scholarspace-hyrax

Hyrax-based repository for GW ETDs and other original content
https://scholarspace.library.gwu.edu/
MIT License
9 stars 4 forks source link

Possible regression on file upload #569

Open dolsysmith opened 1 month ago

dolsysmith commented 1 month ago

I think I caught a regression on the latest GWSS release that breaks PDF uploads. Sheila was getting an error attempting to upload PDF's in the UI, at the point where you add them to the File tab in creating a new work.

So what's going on? My guess is that some other process is resetting the tmp directories to root access, whereas the upload action needs them to be owned by the scholarspace user. Could it be that Bulkrax or the ETD loader is doing this?

kerchner commented 1 month ago

@dolsysmith I don't see any code in the ingest_bulkrax_prep task that would be affecting /opt/scholarspace/scholarspace-hyrax/tmp/cache/hyrax but it is worth observing this folder's ownership before and after running the task.

dolsysmith commented 1 month ago

I did some testing on my dev instance, and when run inside the container, gwss:ingest_pq_etds does create temp folders with the appropriate (scholarspace user) permissions. But I also noticed that in Dan's write-up on the PR, it states, "Run the ingest_bulkrax_prep rake task inside the container (or from the outside using docker exec." If you run the rake task using docker exec and do not provide the --user scholarspace option, the task will be run as root, thus altering the permissions on the temp folder.

@kerchner Do you recall whether you ran this task using Docker exec on test/prod, and if so, whether you might have omitted the --user option? If so, I think that might have caused the issue with the uploads.

If not, I can continue poking.