[PATCH] connect.c: add a way for git-daemon to pass an error back to client
- From
Tom Preston-Werner <tom@github.com>
- Date
- Nov 1, 2008, 01:59 UTC
- Message-ID
- <b97024a40810311859t2e5a6102u31ad4480e7c75c03@mail.gmail.com>
The current behavior of git-daemon is to simply close the connection on any error condition. This leaves the client without any information as to the cause of the failed fetch/push/etc.
This patch allows get_remote_heads to accept a line prefixed with "ERR" that it can display to the user in an informative fashion. Once clients can understand this ERR line, git-daemon can be made to properly report "repository not found", "permission denied", or other errors.
Example
S: ERR No matching repository. C: fatal: remote error: No matching repository.
Signed-off-by: Tom Preston-Werner <tom@github.com> --- connect.c | 3 +++ 1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/connect.c b/connect.c index 0c50d0a..3af91d6 100644 --- a/connect.c +++ b/connect.c @@ -70,6 +70,9 @@ struct ref **get_remote_heads(int in, struct ref **list, if (buffer[len-1] == '\n') buffer[--len] = 0; + if (len > 4 && !memcmp("ERR", buffer, 3)) + die("remote error: %s", buffer + 4); + if (len < 42 || get_sha1_hex(buffer, old_sha1) || buffer[40] != ' ') die("protocol error: expected sha/ref, got '%s'", buffer); name = buffer + 41;
-- 1.6.0.2