[NXDRIVE-2321] [Direct Transfer] Fix file name escaping when checking for document existence on the server - Nuxeo Issue Tracker

XML

Word

Printable

Details

Type: Bug
Status: Resolved
Priority: Critical
Resolution: Fixed
Affects Version/s: 4.4.4
Fix Version/s: 5.2.2
Component/s: Direct Transfer

Release Notes Summary:
File Name Escaping
Release Notes Description:

Hide

We fixed file name escaping when checking for document existence on the server when it could contain the \r character. This is likely to happen on macOS only but the prevention is available to all platforms.

Show
We fixed file name escaping when checking for document existence on the server when it could contain the \r character. This is likely to happen on macOS only but the prevention is available to all platforms.
Epic Link:
Direct Transfer
Tags:
- nxDrive
- platform-review
Sprint:
nxDrive 11.2.6, nxDrive 11.2.22
Story Points:
2

Description

Error

Uploading the file My \r file will not work and put the upload in error:

Failed to invoke operation FileManager.Import, Failed to execute query: SELECT * FROM Document WHERE ecm:parentId = 'b5655fd3-f82e-47f0-954b-df89d8f13794' AND content/name = 'My \r' AND ecm:isTrashed = 0, Lexical Error: Illegal character <\r> at offset 107

Fix

Escape backslashes.
In the dialog box asking confirmation to cancel an upload, the file name should be escaped too, like using repr(filename).

Attachments

Options
- Sort By Name
- Sort By Date
- Ascending
- Descending
- Thumbnails
- List
- Download All

Attachments

tests.patch
0.8 kB
2021-06-02 16:22

Issue Links

Is referenced in

PR for master: #2786

mentioned in: Page Loading...

Activity

People

Assignee:

Mickaël Schoentgen

Reporter:

Mickaël Schoentgen

Participants:

Jenkins, Mickaël Schoentgen

Votes:

0 Vote for this issue

Watchers:

2 Start watching this issue

Dates

Created:

2020-10-07 16:08

Updated:

2021-06-24 08:16

Resolved:

2021-06-07 15:14

Time Tracking

Estimated:

Not Specified

Remaining:

Logged: