Skip to content

Add public/uploads/tmp to allowed upload paths

Jan Provaznik requested to merge jprovazn-fix-form-uploads into master

What does this MR do?

Adds public/uploads/tmp directory to allowed upload paths.

Are there points in the code the reviewer needs to double check?

Why was this MR needed?

When direct_upload is enabled and a form file is being uploaded, then workhorse uses public/uploads/tmp path. If uploads.storage_path i sset to a different directory, then upload fails because public/uploads/tmp is not in allowed paths.

Reproducer:

  • change uploads.storage_path to a custom directory
  • enable direct_uploads
  • got to group settings and try to upload new avatar

Screenshots (if relevant)

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

Closes #49585 (closed)

Edited by Kamil Trzciński

Merge request reports