Last modified: 2012-08-29 23:11:50 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T28209, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 26209 - UploadWizard: uploading file with colon in name leads to error
UploadWizard: uploading file with colon in name leads to error
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
UploadWizard (Other open bugs)
unspecified
All All
: Normal normal (vote)
: ---
Assigned To: Neil Kandalgaonkar
:
Depends on:
Blocks: 27260
  Show dependency treegraph
 
Reported: 2010-12-02 20:21 UTC by Derk-Jan Hartman
Modified: 2012-08-29 23:11 UTC (History)
5 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Derk-Jan Hartman 2010-12-02 20:21:07 UTC
To quote:

 tried uploading a file with a colon (:) in the name on commons; it didn't disallow it, but I got this really unfriendly message:
Internal error
key '2muj1nywb867g7yba09qnnr99h4qos5.' is not in a proper format
Backtrace:
#0 /usr/local/apache/common-local/wmf-deployment/includes/upload/UploadBase.php(557): UploadStash->stashFile('/tmp/phpnXpjgd', Array, NULL)
#1 /usr/local/apache/common-local/wmf-deployment/includes/upload/UploadBase.php(569): UploadBase->stashSessionFile(NULL)
#2 /usr/local/apache/common-local/wmf-deployment/includes/specials/SpecialUpload.php(322): UploadBase->stashSession()
#3 /usr/local/apache/common-local/wmf-deployment/includes/specials/SpecialUpload.php(413): SpecialUpload->showUploadWarning(Array)
#4 /usr/local/apache/common-local/wmf-deployment/includes/specials/SpecialUpload.php(167): SpecialUpload->processUpload()
#5 /usr/local/apache/common-local/wmf-deployment/includes/SpecialPage.php(560): SpecialUpload->execute(NULL)
#6 /usr/local/apache/common-local/wmf-deployment/includes/Wiki.php(254): SpecialPage::executePath(Object(Title))
#7 /usr/local/apache/common-local/wmf-deployment/includes/Wiki.php(64): MediaWiki->handleSpecialCases(Object(Title), Object(OutputPage), Object(WebRequest))
#8 /usr/local/apache/common-local/wmf-deployment/index.php(117): MediaWiki->performRequestForTitle(Object(Title), NULL,
Object(OutputPage), Object(User), Object(WebRequest))
#9 /usr/local/apache/common-local/live-1.5/index.php(3): require('/usr/local/apac...')
#10 {main}
Comment 1 Roan Kattouw 2010-12-02 22:33:32 UTC
(In reply to comment #0)
> key '2muj1nywb867g7yba09qnnr99h4qos5.' is not in a proper format
Presumably this is because guessing the extension failed, and UploadStash tried to append ".$ext" anyway?
Comment 3 magog.the.ogre 2010-12-10 21:42:17 UTC
Another error here: http://en.wikipedia.org/w/index.php?title=Wikipedia:Village_pump_(technical)&oldid=401676326#.22Internal_error.22

Looks like it's for svg's, and has something to do with the key created in conjunction with the filename.
Comment 4 Neil Kandalgaonkar 2011-03-19 06:32:32 UTC
For simplicity, I am disallowing colons client side now, as of r84313.

Presumably this will fix this bug, although I can't replicate it at the moment.
Comment 5 Sage Ross 2012-08-17 21:39:54 UTC
I've just run into this problem, so it doesn't seem to be fixed. When I add a file with a colon in the name to the upload wizard, I get the triangle with the "!" in it, and "Unknown error: "unknown-warning"."

If that's the intended behavior, it should give a more helpful error message.

The files I tried with are .png.
Comment 6 Derk-Jan Hartman 2012-08-29 22:47:19 UTC
@sage the file with the colon. Was that the original title of the file you are uploading in the upload step, or was it the title you were trying to give to the final upload during the details step ?
Comment 7 Sage Ross 2012-08-29 23:00:43 UTC
This was during the upload step. The file name was: "Screenshot from 2012-08-17 17:24:54.png" or similar. (I tried it with a couple of screenshots with similar titles.)
Comment 8 Derk-Jan Hartman 2012-08-29 23:11:50 UTC
OK, confirmed. A badfilename case and it is fixed since this commit https://gerrit.wikimedia.org/r/21698

The fix is not yet live on Commons.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links