Submissions with attachments with unicode filenames are not saved

[yanokwa]

If you have an Enketo file submission with unicode (e.g., é) the submission will go through fine, but when you try to edit the data, you'll see...

In the DB, I see that the blobs have no ID.

central=> select * from submission_attachments where "submissionDefId" = 180867;
 blobId |       name        | submissionDefId | index | isClientAudit
--------+-------------------+-----------------+-------+---------------
  14316 | file1-10_11_1.bin |          180867 |       | f
        | fîlé2-10_11_6.bin |          180867 |       | f
        | f😂le3-10_11_9.bin |          180867 |       | f

central=> select id,"contentType" from blobs where id = null;
 id | contentType
----+-------------
(0 rows)

The form and files I used to do the above are here:

• files.zip
• form.xlsx

This does not happen in Collect because files are renamed to a timestamp before upload.

[yanokwa]

@eyelidlessness This isn't an Enketo bug. I'm tagging you for visibility.

[matthew-white]

Notes for the QA team

This issue can be tested using a form with a media field. This issue only comes up in Enketo, but it'd be great to verify that a submission from Collect also continues to work.

[yanokwa]

@getodk/testers Please be sure to also check special characters like (, _, # and the like.

[dbemke]

I don't know if this is in the scope of this PR when the file name contains a symbol (that shouldn't contain) like $ then while editing the user sees missing image and is able to download the file (which is broken).

[sadiqkhoja]

File is downloadable from the Submission table:

When I click on download icon on submission table, backend receives this request:

/v1/projects/473/forms/build_media_1556624635/submissions/uuid%3Abe2af264-615d-4658-afa5-385158ede558/attachments/%24dollar-8_45_1.jpg

However, URL in Enketo is /-/media/get/1/uuid:be2af264-615d-4658-afa5-385158ede558/%24dollar-8_45_1.jpg and it is not sending any request to backend.

This looks like Enketo issue

[dbemke]

We finished testing other characters so please let us know if the issue above will be fixed as a separate so that we can remove the needs testing label.

[matthew-white]

Yes, that will be fixed separately in Enketo, so we can remove the "needs testing" label here. Thank you for finding that issue!

[srujner]

Tested with Success!

[dbemke]

Tested with Success!