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
F19799254: D21119.id50298.diff
Fri, Feb 27, 11:12 PM
F19665704: D21119.id50298.diff
Feb 7 2026, 1:57 PM
F19607355: D21119.diff
Feb 3 2026, 11:03 AM
F19607353: D21119.diff
Feb 3 2026, 11:03 AM
F19469329: D21119.id50298.diff
Jan 4 2026, 9:02 PM
F19059156: D21119.diff
Nov 29 2025, 4:40 AM
F18904625: D21119.id.diff
Nov 8 2025, 10:14 AM
F18885566: D21119.diff
Nov 7 2025, 3:27 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