If the Conduit client supports gzip, make calls with "Accept-Encoding: gzip"
ClosedPublic
Actions

Authored by epriestley on Apr 14 2020, 11:21 PM.

Tags

None

Referenced Files

	F15560391: D21118.id50295.diff
	Tue, Apr 29, 2:21 PM

	F15545887: D21118.id50295.diff
	Sat, Apr 26, 1:19 PM

	F15545692: D21118.id50294.diff
	Sat, Apr 26, 12:03 PM

	F15537487: D21118.diff
	Thu, Apr 24, 2:05 PM

	F15532530: D21118.id50294.diff
	Wed, Apr 23, 4:07 PM

	F15511147: D21118.diff
	Thu, Apr 17, 12:07 AM

	F15441507: D21118.id.diff
	Mar 26 2025, 6:32 PM

	F15440630: D21118.id50295.diff
	Mar 26 2025, 1:54 PM

View All Files

Subscribers

None

Details

Reviewers: None
Maniphest Tasks: T13507: Compress Conduit client requests
Commits: rARCa77da426af20: If the Conduit client supports gzip, make calls with "Accept-Encoding: gzip"

Summary

Ref T13507. Add "Accept-Encoding: gzip" to requests if we can decompress responses.

When we receive a compressed response, decompress it.

Test Plan

Added debugging code, ran some commands, saw smaller payloads over the wire and inline decompression.

Diff Detail

Repository

rARC Arcanist

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Apr 14 2020, 11:21 PM

Harbormaster completed remote builds in B24112: Diff 50294.Apr 14 2020, 11:22 PM

This revision was not accepted when it landed; it landed in state Needs Review.Apr 14 2020, 11:22 PM

epriestley requested review of this revision.

Closed by commit rARCa77da426af20: If the Conduit client supports gzip, make calls with "Accept-Encoding: gzip" (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rARCa77da426af20: If the Conduit client supports gzip, make calls with "Accept-Encoding: gzip".

Revision Contents
Changeset List

Path

Size

src/

future/

http/

BaseHTTPFuture.php

20 lines

HTTPSFuture.php

19 lines

Diff 50295

View Options

src/future/http/BaseHTTPFuture.php