Last modified: 2014-05-10 11:46:00 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 T36238, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 34238 - [Regression] mwstore:// URLs appear in various error messages
[Regression] mwstore:// URLs appear in various error messages
Status: NEW
Product: MediaWiki
Classification: Unclassified
File management (Other open bugs)
1.20.x
All All
: Normal normal (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-02-07 06:49 UTC by Tim Starling
Modified: 2014-05-10 11:46 UTC (History)
5 users (show)

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


Attachments

Description Tim Starling 2012-02-07 06:49:08 UTC
In trunk after the FileBackend branch merge, many error messages related to files have mwstore:// URLs as message parameters. This risks being even more confusing than the "public" placeholder that I introduced in filerepo, which drew many complaints from users. 

I think replacing the "mwstore://local-backend/local-public" prefix with a simple "..." by default would be less confusing to ordinary humans. For sysadmins, optionally converting the mwstore:// URLs to some backend-specific identifier (for example actual filesystem paths) would be a useful debugging tool.
Comment 1 Tim Starling 2012-02-07 06:51:17 UTC
Milestone 1.19.0 since these errors are mostly seen by non-Wikimedia users.
Comment 2 Aaron Schulz 2012-02-10 23:33:31 UTC
(In reply to comment #0)
> I think replacing the "mwstore://local-backend/local-public" prefix with a
> simple "..." by default would be less confusing to ordinary humans.

Maybe...I'd prefer to keep the container, in case a failed up was moving some file from one zone to another, likely with the same relative paths.

> For sysadmins, optionally converting the mwstore:// URLs to some backend-specific
> identifier (for example actual filesystem paths) would be a useful debugging
> tool.

Can't we just use trigger_error() or something and put more descriptive paths in there (e.g. absolute FS paths or real swift paths)? This would aid in debugging without conflating UI errors with detailed errors for admin debugging.
Comment 3 Aaron Schulz 2012-02-24 20:11:24 UTC
Error logs should be better with r112347.
Comment 4 Rob Lanphier 2012-04-20 18:29:05 UTC
With r112347, this is hopefully good enough for most purposes.  Will fix up right eventually.  Taking off of Aaron since this he probably won't have time for this soon.
Comment 5 Marcin Cieślak 2012-10-11 06:27:18 UTC
I got this when uploading a file on installation close to master (55b5b9f52dad8466749ea43819c1aff23b0044cd) and I got this:

Das Verzeichnis „mwstore://local-backend/local-public/e/e0“ konnte nicht angelegt werden, which is "directorycreateerror".
Comment 6 Andre Klapper 2013-03-20 15:52:26 UTC
Note: As this has the 1.21.0 target milestone set (bumped from 1.20, bumped from 1.19), this would require some final cleanup as written in comment 4. If this does not happen in the next weeks, the Target Milestone will likely get removed.

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


Navigation
Links