Revert "retry request without query when info/refs?query fails"
authorShawn O. Pearce <spearce@spearce.org>
Thu, 20 Sep 2012 05:58:00 +0000 (22:58 -0700)
committerJunio C Hamano <gitster@pobox.com>
Thu, 20 Sep 2012 17:25:21 +0000 (10:25 -0700)
commit6ac964a627e02ac73d4c97c492acf604942030e9
treea79dbc104d7e06f5b75d00b3e20fdf136619cf00
parent25ae7cfd19c8f21721363c64163cd5d9d1135b20
Revert "retry request without query when info/refs?query fails"

This reverts commit 703e6e76a14825e5b0c960d525f34e607154b4f7.

Retrying without the query parameter was added as a workaround
for a single broken HTTP server at git.debian.org[1]. The server
was misconfigured to route every request with a query parameter
into gitweb.cgi. Admins fixed the server's configuration within
16 hours of the bug report to the Git mailing list, but we still
patched Git with this fallback and have been paying for it since.

Most Git hosting services configure the smart HTTP protocol and the
retry logic confuses users when there is a transient HTTP error as
Git dropped the real error from the smart HTTP request. Removing the
retry makes root causes easier to identify.

[1] http://thread.gmane.org/gmane.comp.version-control.git/137609

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
Acked-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
remote-curl.c