Page MenuHomePhabricator

If the Conduit server asserts it has the "gzip" capability, compress requests
ClosedPublic

Authored by epriestley on Apr 14 2020, 11:45 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 22, 7:25 AM
Unknown Object (File)
Wed, Nov 20, 3:14 AM
Unknown Object (File)
Wed, Nov 20, 1:27 AM
Unknown Object (File)
Sat, Nov 16, 3:15 PM
Unknown Object (File)
Tue, Nov 12, 12:02 PM
Unknown Object (File)
Thu, Nov 7, 8:48 PM
Unknown Object (File)
Oct 24 2024, 1:41 AM
Unknown Object (File)
Oct 21 2024, 1:57 AM
Subscribers
None

Details

Summary

Ref T13507. For various messy reasons we can't blindly assume the server supports "gzip" -- but if the server tells us it does, we're on firmer ground.

If the server returns an "X-Conduit-Capabilities: gzip" header and we have compression support locally, compress subsequent requests.

This restores D21073, which was reverted by D21076.

Test Plan

With a gzip-asserting server, added debugging code and ran various "arc" commands. Saw the 2nd..Nth calls hit compression code.

Diff Detail

Repository
rARC Arcanist
Branch
gzip2
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 24114
Build 33202: Run Core Tests
Build 33201: arc lint + arc unit