Don't use "/tmp" folder, when resizing uploaded images

Components

Labels

None

Description

In the "\kUploadHelper::resizeUploadedFile" method instead of using "/tmp" folder (result of "sys_get_temp_dir" function call) use WRITEABLE . '/tmp' path.

Context Information

None

Additional information (do not use)

None

Linked issues

is blocked by

INP-1773

Use "round" function during image resize instead of "ceil"

Activity

Show:

Alex January 4, 2021 at 10:33 AM

alex closed D371: Don't use "/tmp" folder, when resizing uploaded images.
alex updated the diff for D371: Don't use "/tmp" folder, when resizing uploaded images.

Alex January 4, 2021 at 8:33 AM

alex committed rINP16652: Fixes - Don't use "/tmp" folder, when resizing uploaded images.

Fixes - Don't use "/tmp" folder, when resizing uploaded images

Differential Revision: http://qa.in-portal.org/D371

Alex January 4, 2021 at 8:33 AM

alex added "INP-1772" JIRA issue(s) to rINP16652: Fixes - Don't use "/tmp" folder, when resizing uploaded images.

Alex January 4, 2021 at 8:33 AM

alex closed D371: Don't use "/tmp" folder, when resizing uploaded images.
alex updated the diff for D371: Don't use "/tmp" folder, when resizing uploaded images.

[API] Administrator January 4, 2021 at 8:33 AM

User committed a fix to 5.2.x. Commit Message:

Fixes - Don't use "/tmp" folder, when resizing uploaded images

Differential Revision: http://qa.in-portal.org/D371

Fixed

Details
Priority
Minor
Assignee
Alex
Reporter
Alex
Developer
Erik S.
Reviewer
Alex
Change Log Group
Fixed
Change Log Message
When "storage_format" field option was, but uploaded file already had correct dimensions it was deleted (non-uploaded).
Patch Instructions
Patches must be submitted through Phabricator.
To submit patch via Command Line use Patches Workflow (via Arcanist) tutorial.
To submit patch via Web Interface use Patches Workflow (via Web Interface) tutorial.
Time tracking
1m logged
Story Points
1
Fix versions
5.2.2-B3

Created June 26, 2019 at 9:06 AM

Updated December 29, 2024 at 8:49 PM

Resolved January 4, 2021 at 8:33 AM