Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bad URI syntax exception when file name contains '%' percent character #10604

Open
timar opened this issue Nov 26, 2024 · 1 comment
Open

Bad URI syntax exception when file name contains '%' percent character #10604

timar opened this issue Nov 26, 2024 · 1 comment
Labels
bug Something isn't working

Comments

@timar
Copy link
Member

timar commented Nov 26, 2024

Bad URI syntax exception when file name contains '%' percent character. E.g. someone has a file in Nextcloud with the name test%.odt, then it won't open in Collabora Online. In the log we will find the "Bad URI syntax" exception.

We had this problem long ago, fixed it (cf. 898c47f), had protected the fix with unit test. But regression happened nevertheless.

@timar timar added bug Something isn't working unconfirmed labels Nov 26, 2024
@timar
Copy link
Member Author

timar commented Nov 26, 2024

Local make run, I renamed writer-edit.fodt to writer-edit%.fodt.

Error: 500 - Internal Server Error!
Cannot process the request - Bad URI syntax: URI encoding: not a hex digit /browser/e729f7ed07/cool.html
Please contact your system administrator.

The "not a hex digit" message comes from poco, https://github.com/austinsc/Poco/blob/f459e1ddb12354edd5c9908b2d53ec129bdcfa98/Foundation/src/URI.cpp#L617

@timar timar removed the unconfirmed label Nov 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: No status
Development

No branches or pull requests

1 participant