Make WordPress Core

Opened 15 years ago

Closed 15 years ago

Last modified 15 years ago

#8674 closed feature request (fixed)

Support compressed streams in the HTTP API

Reported by: jacobsantos's profile jacobsantos Owned by: jacobsantos's profile jacobsantos
Milestone: 2.8 Priority: normal
Severity: normal Version:
Component: HTTP API Keywords: has-patch needs-testing
Focuses: Cc:

Description

Some of the transports support decompressed compressed data, while the others do not. The ticket establishes that all transports will support decompressed data that was sent over HTTP.

I think the PclZip library can be used, but it will require testing to be sure.

Attachments (2)

8674.diff (5.7 KB) - added by jacobsantos 15 years ago.
Add Gzip support to the HTTP API.
8674.2.diff (7.1 KB) - added by jacobsantos 15 years ago.
Better and complete implementation of gzip.

Download all attachments as: .zip

Change History (12)

#1 @DD32
15 years ago

Ideally PclZip() should be avoided at all costs if there is an alternate decompression function available, Simply for the fact it does everything in PHP rather than Compiled C++, and Compression functions are better optimized in such a language.

See: http://au.php.net/manual/en/function.gzuncompress.php http://au.php.net/manual/en/function.gzdecode.php

#2 @jacobsantos
15 years ago

  • Component changed from General to HTTP

#3 @jacobsantos
15 years ago

  • Summary changed from Support Compressed streams in the HTTP API to Support compressed streams in the HTTP API

@jacobsantos
15 years ago

Add Gzip support to the HTTP API.

#4 @jacobsantos
15 years ago

  • Keywords has-patch needs-testing added

#5 @jacobsantos
15 years ago

The patch implements Gzip classes, but isn't used in the HTTP API anywhere.

#6 @jacobsantos
15 years ago

  • Keywords needs-patch added; has-patch removed

Well, after thinking about and talking to DD32, a better solution would be to use one class instead. Still needs testing though.

@jacobsantos
15 years ago

Better and complete implementation of gzip.

#7 @jacobsantos
15 years ago

  • Keywords has-patch added; needs-patch removed

Patch needs testing. Need to see if the patch works for those having issues.

#8 @westi
15 years ago

(In [10410]) First pass compression support for the HTTP API. See #8674 props jacobsantos.

#9 @jacobsantos
15 years ago

  • Resolution set to fixed
  • Status changed from new to closed

I'm going to say that this works, since it has been tested and appears to work okay. Any problems can be patched in a new ticket.

#10 @Denis-de-Bernardy
15 years ago

See #9761 in particular

Note: See TracTickets for help on using tickets.