Skip to content

Commit

Permalink
Merge branch 'sp/maint-http-info-refs-no-retry'
Browse files Browse the repository at this point in the history
Kills an old workaround for a unlikely server misconfiguration that
hurts debuggability.

* sp/maint-http-info-refs-no-retry:
  Revert "retry request without query when info/refs?query fails"
  • Loading branch information
gitster committed Sep 30, 2012
2 parents 68a31b7 + 6ac964a commit d9c7314
Showing 1 changed file with 2 additions and 16 deletions.
18 changes: 2 additions & 16 deletions remote-curl.c
Expand Up @@ -95,7 +95,7 @@ static struct discovery* discover_refs(const char *service)
struct strbuf buffer = STRBUF_INIT;
struct discovery *last = last_discovery;
char *refs_url;
int http_ret, is_http = 0, proto_git_candidate = 1;
int http_ret, is_http = 0;

if (last && !strcmp(service, last->service))
return last;
Expand All @@ -113,19 +113,6 @@ static struct discovery* discover_refs(const char *service)
refs_url = strbuf_detach(&buffer, NULL);

http_ret = http_get_strbuf(refs_url, &buffer, HTTP_NO_CACHE);

/* try again with "plain" url (no ? or & appended) */
if (http_ret != HTTP_OK && http_ret != HTTP_NOAUTH) {
free(refs_url);
strbuf_reset(&buffer);

proto_git_candidate = 0;
strbuf_addf(&buffer, "%sinfo/refs", url);
refs_url = strbuf_detach(&buffer, NULL);

http_ret = http_get_strbuf(refs_url, &buffer, HTTP_NO_CACHE);
}

switch (http_ret) {
case HTTP_OK:
break;
Expand All @@ -144,8 +131,7 @@ static struct discovery* discover_refs(const char *service)
last->buf_alloc = strbuf_detach(&buffer, &last->len);
last->buf = last->buf_alloc;

if (is_http && proto_git_candidate
&& 5 <= last->len && last->buf[4] == '#') {
if (is_http && 5 <= last->len && last->buf[4] == '#') {
/* smart HTTP response; validate that the service
* pkt-line matches our request.
*/
Expand Down

0 comments on commit d9c7314

Please sign in to comment.