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

File with a space is replaced with + on downloading #11169

Closed
ScharfViktor opened this issue Jul 10, 2024 · 6 comments · Fixed by #11187
Closed

File with a space is replaced with + on downloading #11169

ScharfViktor opened this issue Jul 10, 2024 · 6 comments · Fixed by #11187
Assignees
Labels
Priority:p2-high Escalation, on top of current planning, release blocker Type:Bug Something isn't working

Comments

@ScharfViktor
Copy link
Contributor

Version 6.1.0
Web client version 9.1.0
MacOs 14.4.1

Steps:

  • create file new file.odt (with space)
  • download file

Expected: file is downloaded as new file.odt

Actual: file is downloaded as new+file.odt
Screenshot 2024-07-10 at 09 32 41

@ScharfViktor ScharfViktor added Type:Bug Something isn't working Priority:p2-high Escalation, on top of current planning, release blocker labels Jul 10, 2024
@ScharfViktor ScharfViktor moved this from Qualification to Prio 2 in Infinite Scale Team Board Jul 10, 2024
@JammingBen
Copy link
Contributor

JammingBen commented Jul 10, 2024

Might be a fallout from cs3org/reva#4748 😬

Edit: fix here: cs3org/reva#4762

@JammingBen JammingBen self-assigned this Jul 10, 2024
@JammingBen JammingBen moved this from Prio 2 to In progress in Infinite Scale Team Board Jul 10, 2024
@ScharfViktor
Copy link
Contributor Author

@JammingBen please adapt the e2e download.feature when you fix this bug as https://github.com/owncloud/web/compare/downloadFileWithSpace?expand=1

We could have found this degradation if we had a test for it, but we didn't.

Screenshot 2024-07-10 at 10 11 08

@ScharfViktor
Copy link
Contributor Author

ScharfViktor commented Jul 10, 2024

file with ? : * | \ ; <>-> changed as _
image

@JammingBen
Copy link
Contributor

file with ? : * | \ ; <>-> changed as _

This "issue" is not new though, and I'm not sure if there is anything we can do about it. The content-disposition header is being encoded correctly, so this seems to be a browser thing. @kulmann How do you feel about this, acceptable?

@kulmann
Copy link
Contributor

kulmann commented Jul 15, 2024

file with ? : * | \ ; <>-> changed as _

This "issue" is not new though, and I'm not sure if there is anything we can do about it. The content-disposition header is being encoded correctly, so this seems to be a browser thing. @kulmann How do you feel about this, acceptable?

That's acceptable. As long as blanks work correctly again. 👍

@JammingBen
Copy link
Contributor

Fixed via the current oCIS rolling release.

See #11187 for the e2e test adjustment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority:p2-high Escalation, on top of current planning, release blocker Type:Bug Something isn't working
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants