Paths

Table of Contentst

Differential D20745

Allow HTTPSFuture to stream very large files directly to disk
ClosedPublic
Actions

Authored by epriestley on Aug 29 2019, 5:38 PM.

Tags

None

Referenced Files

	F15469601: D20745.id49465.diff
	Fri, Apr 4, 2:58 PM

	F15456953: D20745.id49465.diff
	Sun, Mar 30, 1:17 PM

	F15452576: D20745.id49462.diff
	Sat, Mar 29, 6:05 AM

	F15450550: D20745.id.diff
	Fri, Mar 28, 4:25 PM

	F15446319: D20745.diff
	Thu, Mar 27, 6:22 PM

	F15438319: D20745.id.diff
	Wed, Mar 26, 12:08 AM

	F15426709: D20745.id49465.diff
	Sun, Mar 23, 10:32 AM

	F15416271: D20745.diff
	Thu, Mar 20, 10:12 AM

Subscribers

None

Details

Reviewers: None
Maniphest Tasks: T13396: Support downloads of 2GB+ files with HTTPSFuture
Commits: rPHUdb94ec322abf: (stable) Allow HTTPSFuture to stream very large files directly to disk
rPHU45c553b5c1d5: Allow HTTPSFuture to stream very large files directly to disk

Summary

Ref T13396. Support a mode which directs "HTTPSFuture" to stream response bodies to disk. In this mode, use the new streaming parser to break past the 2GB limit.

Test Plan

Ran various HTTP commands to attempt to establish that this does not change any existing behavior.
In the next change (internal), modified bin/host download to use this pathway and downloaded large files.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Aug 29 2019, 5:38 PM

Harbormaster completed remote builds in B23321: Diff 49462.Aug 29 2019, 5:38 PM

epriestley requested review of this revision.Aug 29 2019, 5:38 PM

This revision was not accepted when it landed; it landed in state Needs Review.Aug 29 2019, 7:06 PM

Closed by commit rPHU45c553b5c1d5: Allow HTTPSFuture to stream very large files directly to disk (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rPHU45c553b5c1d5: Allow HTTPSFuture to stream very large files directly to disk.

epriestley added a commit: rPHUdb94ec322abf: (stable) Allow HTTPSFuture to stream very large files directly to disk.

Revision Contents
Changeset List

Path

Size

src/

future/

http/

HTTPSFuture.php

105 lines

parser/

http/

PhutilHTTPResponse.php

24 lines

PhutilHTTPResponseParser.php

18 lines

Diff 49465

src/future/http/HTTPSFuture.php

Loading...

src/parser/http/PhutilHTTPResponse.php

Loading...

src/parser/http/PhutilHTTPResponseParser.php

Loading...