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

Re: [PATCH] lessen the impression of unexpectedness on remote hangup

From
Heiko Voigt <hvoigt@hvoigt.net>
Date
Jun 13, 2012, 21:28 UTC
Message-ID
<20120613212805.GA19146@book.hvoigt.net>
In-Reply-To
<20120611190207.GA20889@sigill.intra.peff.net>
On Mon, Jun 11, 2012 at 03:02:07PM -0400, Jeff King wrote:
Show 28 quoted lines
> On Sun, Jun 10, 2012 at 08:23:10PM +0200, Heiko Voigt wrote:
> 
> > If a server accessed through ssh is denying access git will currently
> > issue the message
> > 
> > 	"fatal: The remote end hung up unexpectedly"
> > 
> > as the last line. This sounds as if something really ugly just happened.
> > Since this is a quite typical situation in which users regularly get
> > lets just say:
> > 
> > 	"fatal: The remote end hung up"
> > 
> > Signed-off-by: Heiko Voigt <hvoigt@hvoigt.net>
> > ---
> > We just got this bug report in msysgit
> > 
> > https://github.com/msysgit/msysgit/issues/28
> > 
> > and IIRC there have been more people questioning that line.
> 
> This does not seem like it would lessen the surprise all that much. I
> wonder if we could use some context about where we are in the protocol
> to tell more. For example, if the remote end hangs up before advertising
> its refs, it is probably an authentication error or a missing
> repository. And we should say that. If it happens during ref negotiation
> or during the pack file, then it really is unexpected; the other end has
> broken protocol, and it probably makes sense to say so.

I agree that would be better. I will have a look if I can cook something up. In the meantime I think changing the message is better than unnecessarily worrying the user.

Cheers Heiko
Previous: Jeff KingNext: Heiko Voigt
Message 5 of 13 in “lessen the impression of unexpectedness on remote hangup”
  1. lessen the impression of unexpectedness on remote hangupHeiko Voigt, Jun 10, 2012
  2. Jonathan NiederJun 10, 2012
  3. Junio C HamanoJun 11, 2012
  4. Jeff KingJun 11, 2012
  5. Heiko VoigtJun 13, 2012
  6. remove the impression of unexpectedness when access is deniedHeiko Voigt, Jun 14, 2012
  7. Heiko VoigtJun 14, 2012
  8. Junio C HamanoJun 14, 2012
  9. Heiko VoigtJun 14, 2012
  10. remove the impression of unexpectedness when access is deniedHeiko Voigt, Jun 19, 2012
  11. Jonathan NiederMay 4, 2013
  12. Junio C HamanoMay 6, 2013
  13. Jeff KingMay 7, 2013

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.