[bug] Can't upload an image with an ASCII colon (:) in the name #610

Closed
opened 2023-08-03 15:57:04 +00:00 by yukijoou · 2 comments
Contributor

Your setup

From source

Extra details

Using Debian 12 + nginx, nginx config available here https://paste.debian.net/1287796/

Version

stable @ 9d7c877de0

PostgreSQL version

15

What were you trying to do?

Uploading a file with a colon (:) in its name

What did you expect to happen?

The upload works, and the image is visible

What actually happened?

The upload seems to work, but the image doesn't load in the front end, and I get an error in the syslogs

Logs

[error] Internal server error: %Plug.Static.InvalidPathError{message: "invalid path for static asset: /media/c0551660-73d6-4baf-bb02-834fe8a62153/2023-07-28_21%3A11%3A11.png", plug_status: 400}

Severity

I cannot use it as easily as I'd like

Have you searched for this issue?

  • I have double-checked and have not found this issue mentioned anywhere.
### Your setup From source ### Extra details Using Debian 12 + nginx, nginx config available here https://paste.debian.net/1287796/ ### Version stable @ 9d7c877de049303a4d4d8f5ecaac7da94ca59ebb ### PostgreSQL version 15 ### What were you trying to do? Uploading a file with a colon (`:`) in its name ### What did you expect to happen? The upload works, and the image is visible ### What actually happened? The upload seems to work, but the image doesn't load in the front end, and I get an error in the syslogs ### Logs ```shell [error] Internal server error: %Plug.Static.InvalidPathError{message: "invalid path for static asset: /media/c0551660-73d6-4baf-bb02-834fe8a62153/2023-07-28_21%3A11%3A11.png", plug_status: 400} ``` ### Severity I cannot use it as easily as I'd like ### Have you searched for this issue? - [x] I have double-checked and have not found this issue mentioned anywhere.
yukijoou added the
bug
label 2023-08-03 15:57:04 +00:00

this should be fixed by the latest update which enables filename anonymisation by default, let me know if this persists through the next stable

this should be fixed by the latest update which enables filename anonymisation by default, let me know if this persists through the next stable
Member

fixed via 0ec62acb9d
(URLs will never contain colons or other chars Phoenix (plugs) deem invalid)

fixed via 0ec62acb9dc1c6500033086b46c37adefb700c62 *(URLs will never contain colons or other chars Phoenix (plugs) deem invalid)*
Sign in to join this conversation.
No milestone
No project
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: AkkomaGang/akkoma#610
No description provided.