Last modified: 2014-07-26 06:21:25 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 T51371, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 49371 - GitBlit main page sets Cache-Control: no-cache
GitBlit main page sets Cache-Control: no-cache
Status: NEW
Product: Wikimedia
Classification: Unclassified
Git/Gerrit (Other open bugs)
wmf-deployment
All All
: Normal normal (vote)
: ---
Assigned To: Nobody - You can work on this!
https://code.google.com/p/gitblit/iss...
: upstream
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-06-09 22:46 UTC by Ori Livneh
Modified: 2014-07-26 06:21 UTC (History)
11 users (show)

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


Attachments

Description Ori Livneh 2013-06-09 22:46:27 UTC
We should leverage browser caching and a reverse proxy to make GitBlit responsive.  It currently marks all responses as no-cache, which is too aggressive.


$ curl -I "https://git.wikimedia.org/"
HTTP/1.1 200 OK
Date: Sun, 09 Jun 2013 22:45:31 GMT
Server: Jetty(7.6.8.v20121106)
Content-Type: text/html;charset=UTF-8
Content-Language: en-US
Pragma: no-cache
Cache-Control: no-cache, max-age=0, must-revalidate
Content-Length: 1966389
Vary: Accept-Encoding
Comment 1 Chad H. 2013-06-09 23:05:19 UTC
We are currently running it behind Apache as a reverse proxy. I'm fine with trying to improve caching here. If swapping Apache for $somethingElse is necessary, I'm fine with that too.
Comment 2 Chad H. 2013-07-17 23:27:10 UTC
Filed upstream: https://code.google.com/p/gitblit/issues/detail?id=274
Comment 3 Gerrit Notification Bot 2013-07-23 17:45:03 UTC
Change 75366 had a related patch set uploaded by Demon:
Allow the user's browser to cache resources

https://gerrit.wikimedia.org/r/75366
Comment 4 Gerrit Notification Bot 2013-07-23 19:09:02 UTC
Change 75366 merged by Ryan Lane:
Allow the user's browser to cache resources

https://gerrit.wikimedia.org/r/75366
Comment 5 Chad H. 2013-07-23 23:11:49 UTC
This is now deployed with a default expiry of 5 minutes. Look good to you Ori?
Comment 6 Ori Livneh 2013-08-14 09:08:09 UTC
5 minutes seems too short -- it should be a week or more for some static assets --  and I don't think "Cache-Control: private, must-revalidate" is appropriate for an unauthenticated, read-only view of Git data.
Comment 7 Chad H. 2013-08-14 16:34:44 UTC
5 minutes was mostly for testing. That being said, I think this setting affects too many things and not just resources. Maybe we need to hack it a bit.

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


Navigation
Links