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

Re: [RFC PATCH] Add WEBDAV timeout to http-fetch.

From
Junio C Hamano <junkio@cox.net>
Date
Oct 8, 2006, 04:52 UTC
Message-ID
<7vbqonpfyl.fsf@assigned-by-dhcp.cox.net>
In-Reply-To
<Pine.LNX.4.63.0610080034490.14200@wbgn013.biozentrum.uni-wuerzburg.de>
Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:
Show 5 quoted lines
> Actually, I do not think that anybody in her right mind would set this to 
> different values for different repos or servers.
>
> I _know_ that if I hit that very problem, the next thing I'd do is set the 
> timeout to 5 seconds _globally_.
Let's step back a bit.

The DAV request in question the one to remote_ls() in http-fetch.c, which tries to read directly from objects/pack/ instead of using objects/info/packs, and it does not matter if DAV request fails because it would fall back to non-DAV anyway.

Using DAV, if it works with the server, has the advantage of not having to keep objects/info/packs up-to-date from repository owner's point of view. But the repository owner ends up keeping up-to-date as a side effect of keeping info/refs up-to-date anyway (as I do not see a code to read that information over DAV), so there is no point doing this over DAV in practice.

Perhaps we should remove call to remote_ls() from fetch_indices() unconditionally, not just protected with NO_EXPAT and be done with it?

Previous: Johannes SchindelinNext: Johannes Schindelin
Message 11 of 19 in “Problem cloning packed-and-pruned http repository”
  1. Panagiotis IssarisOct 6, 2006
  2. Panagiotis IssarisOct 6, 2006
  3. SeanOct 6, 2006
  4. Add WEBDAV timeout to http-fetch.Sean, Oct 7, 2006
  5. Junio C HamanoOct 7, 2006
  6. Jakub NarebskiOct 7, 2006
  7. Johannes SchindelinOct 7, 2006
  8. Petr BaudisOct 7, 2006
  9. Petr BaudisOct 7, 2006
  10. Johannes SchindelinOct 7, 2006
  11. Junio C HamanoOct 8, 2006
  12. Johannes SchindelinOct 8, 2006
  13. SeanOct 8, 2006
  14. Junio C HamanoOct 8, 2006
  15. Panagiotis IssarisOct 11, 2006
  16. Johannes SchindelinOct 11, 2006
  17. Panagiotis IssarisOct 7, 2006
  18. Junio C HamanoOct 7, 2006
  19. Panagiotis IssarisOct 7, 2006

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.