git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH 0/2] git+curl 8.7.0 workaround

From
Jeff King <peff@peff.net>
Date
Apr 2, 2024, 20:02 UTC
Message-ID
<20240402200254.GA874754@coredump.intra.peff.net>
In-Reply-To
<2n7sn76-p413-5632-4o2s-o5n2p1rqnr5@unkk.fr>
On Sat, Mar 30, 2024 at 09:54:02AM +0100, Daniel Stenberg wrote:
Show 11 quoted lines
> On Fri, 29 Mar 2024, Jeff King wrote:
> 
> > I noticed some http-related failures in the test suite on my Debian
> > unstable system, which recently got an upgraded curl package. It looks
> > like it's related to cases where we use the remote-curl helper in
> > "connect" mode (i.e., protocol v2) and the http buffer is small
> > (requiring us to stream the data to curl). Besides just running t5551,
> > an easy reproduction is:
> 
> This smells like a libcurl regression to me. I "imported" this into our
> issue tracker here: https://github.com/curl/curl/issues/13229

This was all resolved in that issue, but just to summarize for the list here: it was a regression in curl and there's a fix already. Thanks Daniel for your (as usual) prompt digging into the problem (and likewise to Stefan for the actual fix).

Ultimately the issue will be fixed by moving to a different version of libcurl, but here's an easy workaround in the meantime, with a small doc cleanup I found along the way.

  [1/2]: http: reset POSTFIELDSIZE when clearing curl handle
  [2/2]: INSTALL: bump libcurl version to 7.21.3
 INSTALL | 2 +-
 http.c  | 1 +
 2 files changed, 2 insertions(+), 1 deletion(-)
-Peff
Previous: Daniel StenbergNext: Jeff King
Message 3 of 15 in “tests broken with curl-8.7.0”
  1. Jeff KingMar 30, 2024
  2. Daniel StenbergMar 30, 2024
  3. 0/2 git+curl 8.7.0 workaroundJeff King, Apr 2, 2024
  4. 1/2 http: reset POSTFIELDSIZE when clearing curl handleJeff King, Apr 2, 2024
  5. Junio C HamanoApr 2, 2024
  6. Jeff KingApr 3, 2024
  7. Patrick SteinhardtApr 3, 2024
  8. Patrick SteinhardtApr 3, 2024
  9. Jeff KingApr 3, 2024
  10. 2/2 INSTALL: bump libcurl version to 7.21.3Jeff King, Apr 2, 2024
  11. rsbecker@nexbridge.comApr 2, 2024
  12. Jeff KingApr 2, 2024
  13. 3/2 remote-curl: add Transfer-Encoding header only for older curlJeff King, Apr 5, 2024
  14. Daniel StenbergApr 5, 2024
  15. Junio C HamanoApr 5, 2024

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.