Last modified: 2013-12-13 20:14:36 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 T58282, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 56282 - Return gzipped html from web service
Return gzipped html from web service
Status: RESOLVED FIXED
Product: Parsoid
Classification: Unclassified
Web API (Other open bugs)
unspecified
All All
: High normal
: ---
Assigned To: Gabriel Wicke
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-10-28 23:14 UTC by Gabriel Wicke
Modified: 2013-12-13 20:14 UTC (History)
1 user (show)

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


Attachments

Description Gabriel Wicke 2013-10-28 23:14:17 UTC
According to https://www.varnish-cache.org/docs/3.0/tutorial/compression.html Varnish won't compress content on its own, so we should return gzipped HTML if the Accept-Encoding header contains 'gzip'.
Comment 1 Gabriel Wicke 2013-10-31 21:42:10 UTC
Patch in gerrit: https://gerrit.wikimedia.org/r/#/c/92611/
Comment 2 Gabriel Wicke 2013-10-31 21:44:47 UTC
And merged
Comment 3 Gabriel Wicke 2013-11-25 19:26:34 UTC
Reopening, as we disabled gzip compression as we ran into issues in production.
Comment 4 Gerrit Notification Bot 2013-11-25 23:26:36 UTC
Change 97647 had a related patch set uploaded by GWicke:
WIP Bug 56282: Gzip all Parsoid HTML before storing it in Varnish

https://gerrit.wikimedia.org/r/97647
Comment 5 Gerrit Notification Bot 2013-11-25 23:57:34 UTC
Change 97652 had a related patch set uploaded by GWicke:
Bug 56282: Enable gzip compression support, attempt 2

https://gerrit.wikimedia.org/r/97652
Comment 6 Gerrit Notification Bot 2013-11-27 17:27:13 UTC
Change 97652 merged by jenkins-bot:
Bug 56282: Enable gzip compression support, the sequel

https://gerrit.wikimedia.org/r/97652
Comment 7 Gabriel Wicke 2013-12-03 22:18:22 UTC
Scheduled for production deploy tomorrow.
Comment 8 Gerrit Notification Bot 2013-12-03 23:30:54 UTC
Change 97647 abandoned by GWicke:
WIP Bug 56282: Gzip all Parsoid HTML before storing it in Varnish

Reason:
No longer needed now that Parsoid does the gzipping.

https://gerrit.wikimedia.org/r/97647
Comment 9 Gerrit Notification Bot 2013-12-13 19:53:44 UTC
Change 101268 had a related patch set uploaded by GWicke:
Merge "Bug 56282: Enable gzip compression support, the sequel"

https://gerrit.wikimedia.org/r/101268
Comment 10 Gerrit Notification Bot 2013-12-13 19:55:05 UTC
Change 101316 had a related patch set uploaded by GWicke:
Bug 56282: Enable gzip compression support, the sequel

https://gerrit.wikimedia.org/r/101316
Comment 11 Gerrit Notification Bot 2013-12-13 19:57:28 UTC
Change 101268 merged by GWicke:
Merge "Bug 56282: Enable gzip compression support, the sequel"

https://gerrit.wikimedia.org/r/101268
Comment 12 Gerrit Notification Bot 2013-12-13 19:57:30 UTC
Change 101316 merged by GWicke:
Bug 56282: Enable gzip compression support, the sequel

https://gerrit.wikimedia.org/r/101316

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


Navigation
Links