Last modified: 2012-12-12 23:53: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 T44989, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 42989 - Gerrit: show link for raw file content in gerrit "diff" views (a solution is proposed inside)
Gerrit: show link for raw file content in gerrit "diff" views (a solution is ...
Status: RESOLVED INVALID
Product: Wikimedia
Classification: Unclassified
Git/Gerrit (Other open bugs)
wmf-deployment
All All
: Normal enhancement (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-12-12 08:23 UTC by T. Gries
Modified: 2012-12-12 23:53 UTC (History)
3 users (show)

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


Attachments

Description T. Gries 2012-12-12 08:23:18 UTC
Ryan K. wrote:

"I found a solution to the problem:

If a gerrit administrator declares the mimetypes of the files to be safe they will be displayed in-browser rather than downloaded as zip files:
https://gerrit-review.googlesource.com/Documentation/config-gerrit.html#_a_id_mimetype_a_section_mimetype

Could someone edit the gerrit.config file to declare php, javascript, and css files as 'safe'?"


see http://lists.wikimedia.org/pipermail/wikitech-l/2012-December/065013.html
Comment 1 Chad H. 2012-12-12 12:50:00 UTC
As I said on list, this config option doesn't behave the way you guys think it does.

Marking INVALID since there's nothing we can do--please file a bug upstream: https://code.google.com/p/gerrit/issues/list
Comment 2 Ryan Kaldari 2012-12-12 23:53:50 UTC
Chad says he's going to test the config change just to see if it works. If it doesn't, I'll follow up with the Gerrit developers.

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


Navigation
Links