RE: [PATCH 1/2] remote-curl: accept all encoding supported by curl

2c99dd1790529cc4b7e029b1e9dfcc817d18erefs/heads/master Kind regards, Anton Golubev -Original Message- From: Jonathan Nieder [mailto:jrnie...@gmail.com] Sent: Dienstag, 22. Mai 2018 02:01 To: Brandon Williams <bmw...@google.com> Cc: git@vger.kernel.org; Anton Golubev <

Re: [PATCH 1/2] remote-curl: accept all encoding supported by curl

On 05/22, Daniel Stenberg wrote: > On Mon, 21 May 2018, Jonathan Nieder wrote: > > > > Looking at the code here, this succeeds if enough memory is available. > > > There is no check if the given parameter is part of > > > Curl_all_content_encodings(); > > > > By "this" are you referring to the

Re: [PATCH 1/2] remote-curl: accept all encoding supported by curl

On Mon, 21 May 2018, Jonathan Nieder wrote: Looking at the code here, this succeeds if enough memory is available. There is no check if the given parameter is part of Curl_all_content_encodings(); By "this" are you referring to the preimage or the postimage? Are you suggesting a change in

Re: [PATCH 1/2] remote-curl: accept all encoding supported by curl

Hi, Stefan Beller wrote: > On Mon, May 21, 2018 at 4:40 PM, Brandon Williams wrote: >> Configure curl to accept all encoding which curl supports instead of >> only accepting gzip responses. > > This partially reverts aa90b9697f9 (Enable info/refs gzip decompression > in HTTP

Re: [PATCH 1/2] remote-curl: accept all encoding supported by curl

On Mon, May 21, 2018 at 4:40 PM, Brandon Williams wrote: > Configure curl to accept all encoding which curl supports instead of > only accepting gzip responses. This partially reverts aa90b9697f9 (Enable info/refs gzip decompression in HTTP client, 2012-09-19), as that

Re: [PATCH 1/2] remote-curl: accept all encoding supported by curl

Hi, Brandon Williams wrote: > Subject: remote-curl: accept all encoding supported by curl nit: s/encoding/encodings > Configure curl to accept all encoding which curl supports instead of > only accepting gzip responses. Likewise. > This is necessary to fix a bug when using an installation of