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

Re: What's cooking in git.git (Dec 2009, #02; Sat, 05)

From
Tay Ray Chuan <rctay89@gmail.com>
Date
Dec 8, 2009, 05:58 UTC
Message-ID
<be6fef0d0912072158g1334aaf0s46299d833f694f1d@mail.gmail.com>
In-Reply-To
<20091207153736.GC17173@spearce.org>
Hi,
On Mon, Dec 7, 2009 at 11:37 PM, Shawn O. Pearce <spearce@spearce.org> wrote:
Show 23 quoted lines
> Martin Storsj? <martin@martin.st> wrote:
>> On Sun, 6 Dec 2009, Junio C Hamano wrote:
>> >
>> > * tr/http-updates (2009-12-01) 3 commits
>> >  - Allow curl to rewind the RPC read buffer
>> >  - Add an option for using any HTTP authentication scheme, not only basic
>> >  - http: maintain curl sessions
>> >
>> > There was a discussion on a better structure not to require rewinding in
>> > the first place?  I didn't follow it closely...
>>
>> I think the conclusion is: Rewinding support isn't strictly necessary,
>> there's a number of mechanisms in both git and curl that should make sure
>> that those cases shouldn't surface. A few of them in curl have an
>> unfortunate conincidence of bugs up until the latest version, though,
>> leaving much fewer mechanisms in place to avoid this.
>>
>> Since that patch is quite non-intrusive I think it's a good safeguard,
>> though. What do you think, Tay, keep it or leave it?
>
> I think the conclusion of the thread was that what you have queued
> in tr/http-updates is OK as-is.  The patch to grow the postbuffer
> to store the entire request wasn't a good idea and got dropped.

Martin, sorry the late reply. I agree with Shawn. Perhaps we could relook at rewinding-to-any-position when restarting the rpc client (eg. git-send-pack) has been implemented.

-- 
Cheers,
Ray Chuan
Previous: Junio C Hamano
Message 5 of 5 in “What's cooking in git.git (Dec 2009, #02; Sat, 05)”
  1. Junio C HamanoDec 6, 2009
  2. Martin StorsjöDec 6, 2009
  3. Shawn O. PearceDec 7, 2009
  4. Junio C HamanoDec 7, 2009
  5. Tay Ray ChuanDec 8, 2009

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.