Differential D21076

Revert "Compress requests from the Conduit client to Phabricator"
ClosedPublic
Actions

Authored by epriestley on Apr 9 2020, 7:22 PM.

Tags

None

Referenced Files

	F15335024: D21076.id.diff
	Sat, Mar 8, 1:50 PM

	F15335023: D21076.diff
	Sat, Mar 8, 1:50 PM

	F15302770: D21076.diff
	Wed, Mar 5, 9:28 PM

	Unknown Object (File)
	Feb 9 2025, 8:52 AM

	Unknown Object (File)
	Feb 9 2025, 8:52 AM

	Unknown Object (File)
	Feb 9 2025, 8:52 AM

	Unknown Object (File)
	Feb 9 2025, 8:52 AM

	Unknown Object (File)
	Jan 25 2025, 7:30 PM

Subscribers

None

Details

Reviewers: None
Commits: rARCf56c6bde2b45: Revert "Compress requests from the Conduit client to Phabricator"

Summary

This reverts commit 7e25288f496887a78d1ba4a54753c2cca09a6336. See T13507 for discussion, shortly.

This needs to be switched to capability detection because some server configurations can not actually accept these requests.

Test Plan

Straight revert.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Apr 9 2020, 7:22 PM

Harbormaster completed remote builds in B24023: Diff 50205.Apr 9 2020, 7:22 PM

epriestley requested review of this revision.Apr 9 2020, 7:22 PM

This revision was not accepted when it landed; it landed in state Needs Review.Apr 9 2020, 7:25 PM

Closed by commit rARCf56c6bde2b45: Revert "Compress requests from the Conduit client to Phabricator" (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rARCf56c6bde2b45: Revert "Compress requests from the Conduit client to Phabricator".

epriestley mentioned this in D21119: If the Conduit server asserts it has the "gzip" capability, compress requests.Apr 14 2020, 11:45 PM

epriestley mentioned this in rARC377ed2ed8db7: If the Conduit server asserts it has the "gzip" capability, compress requests.Apr 14 2020, 11:51 PM

Revision Contents
Changeset List

Path

Size

src/

conduit/

ConduitClient.php

15 lines

Diff 50206

src/conduit/ConduitClient.php

Loading...