GNOME Bugzilla – Bug 697256
Can't attach files with ":" or spaces in their names
Last modified: 2013-07-15 15:49:24 UTC
Created attachment 240584 [details] twitter webapp webkitgtk3-1.11.92-1.fc19.x86_64 epiphany-3.8.0-1.fc19.x86_64 Happened on both Twitter and in this bugzilla, trying to attach a screenshot with ":" in the name. Unescaped ":" in filename passed to webkit?
It might be a dupe of bug #696639. Xan, do you remember if you tried to upload a file with : in the filename? or does it fail in twitter with any file?
(In reply to comment #1) > It might be a dupe of bug #696639. Likely > Xan, do you remember if you tried to upload > a file with : in the filename? It also fails with spaces in the filename. > or does it fail in twitter with any file? Twitter and bugzilla worked for me once I tried to attach files that didn't have any characters that needed escaping.
*** Bug 697831 has been marked as a duplicate of this bug. ***
*** Bug 702582 has been marked as a duplicate of this bug. ***
If bug #702582 is truly a duplicate: for the record, whitespaces are also causing this problem, as attachment #247188 [details] demonstrates.
Bug #702582 is almost surely a duplicate; if you rename your Bugzilla attachment to something without colons and spaces, it will attach fine and you won't see the "can't show website" screen. By the way, this regression appeared in Epiphany 3.8 or WebKit 2.
Created attachment 248984 [details] test
It seems I can't reproduce the issue
hmm, in other bug reports people try to create a new bug with an attachment? Can you guys reproduce the issue when attaching a file to an existing report?
I can still reproduce the problem (trying to attach a screenshot I just took to twitter). epiphany-3.8.2-1.fc19.x86_64 libwebkit2gtk-2.0.3-1.fc19.x86_64 The problem is most likely in WebKit itself, as the file chooser is spawned off by WebKit, not epiphany.
Created attachment 249003 [details] test test test
(In reply to comment #8) > It seems I can't reproduce the issue Using the same software versions as Bastien, I was also able to upload this test file. However I cannot upload a file named "test test" with a space instead of a colon.
Created attachment 249006 [details] test with space
Space instead of semicolon worked for me too.
(In reply to comment #14) > Space instead of semicolon worked for me too. This seems to be broken in stable but fixed in master. I verified this on a jhbuilt Epiphany and WebKit2 from roughly a week ago and the software versions Bastien posted above. (I uploaded "test test" to another site since my jhbuilt Epiphany doesn't handle https.) Normally I'd pull out the git bisect hammer at this point to figure out if there's a commit we could pick to stable. But <grumble WebKit compilation times>.
I'll try again with the stable branch next week and will try to find the commit that fixed it.
Created attachment 249209 [details] Test with spaces using patched wk stable branch
Ok, found the issue, it was a buggy implementation of KURL::fileSystemPath() used in GTK+ port that has been replaced in master by a better cross-platform solution. I'm going to close this as not gnome and cherrypick the fixes in the stable branch. Thank you all for your help.