threads / patch / 30406

patch, 6 partsRe-roll of http-proxy-more branch

Subject: [PATCH 0/6] Re-roll of http-proxy-more branch

## tl;dr

One message between May 3, 2012 and May 3, 2012. Diffs are folded; open one to read it.

replies: 0people: 1as markdown or json

Nelson Benitez Leon· May 3, 2012, 16:39 UTC · lore

Hi, this is a reroll of http-proxy-more branch melting the fixup patches and taken into account following Jeff's comments:

http://article.gmane.org/gmane.comp.version-control.git/195443

But I kept using specific curl options for proxyusername and password as they proper handle the username to contain special characters like colon as explained here:

http://curl.haxx.se/libcurl/c/curl_easy_setopt.html#CURLOPTPROXYUSERNAME

and also setting those in set_proxy_auth() function makes for cleaner code imho, but will change it if you still feel uncomfortable with that.

Looking forward to having this merged..
Thanks!
Jeff King (1):
  http: fix http_proxy specified without protocol part
Nelson Benitez Leon (5):
  http: try http_proxy env var when http.proxy config option is not set
  http: handle proxy proactive authentication
  http: handle proxy authentication failure (error 407)
  http: Avoid limit of retrying request only twice
  http: rename HTTP_REAUTH to HTTP_AUTH_RETRY
 credential.c  |   12 ++++--
 http-push.c   |   24 ++++++------
 http-walker.c |    2 +-
 http.c        |  111 ++++++++++++++++++++++++++++++++++++++++++++++++++------
 http.h        |    4 +-
 remote-curl.c |    4 +-
 6 files changed, 124 insertions(+), 33 deletions(-)
-- 
1.7.7.6

← back to recent threads