* [PATCH 0/3] Reduced re-roll of http-proxy-more branch
@ 2012-05-11 13:13 Nelson Benitez Leon
2012-05-11 17:00 ` Jeff King
0 siblings, 1 reply; 2+ messages in thread
From: Nelson Benitez Leon @ 2012-05-11 13:13 UTC (permalink / raw)
To: git; +Cc: Jeff King
Hi, based on last feedback by Junio and Jeff I've created this
reduced version of http-proxy-more branch with all the bits
about "reading proxy from environment" removed. When cURL adds
the new api we will only need to expand the code introduced by
commit 1/3 .
Regards,
Nelson Benitez Leon (3):
http: handle proxy authentication failure (error 407)
http: Avoid limit of retrying request only twice
http: rename HTTP_REAUTH to HTTP_AUTH_RETRY
http.c | 37 ++++++++++++++++++++++++++++++++-----
http.h | 2 +-
2 files changed, 33 insertions(+), 6 deletions(-)
--
1.7.7.6
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH 0/3] Reduced re-roll of http-proxy-more branch
2012-05-11 13:13 [PATCH 0/3] Reduced re-roll of http-proxy-more branch Nelson Benitez Leon
@ 2012-05-11 17:00 ` Jeff King
0 siblings, 0 replies; 2+ messages in thread
From: Jeff King @ 2012-05-11 17:00 UTC (permalink / raw)
To: Nelson Benitez Leon; +Cc: git
On Fri, May 11, 2012 at 03:13:25PM +0200, Nelson Benitez Leon wrote:
> Hi, based on last feedback by Junio and Jeff I've created this
> reduced version of http-proxy-more branch with all the bits
> about "reading proxy from environment" removed. When cURL adds
> the new api we will only need to expand the code introduced by
> commit 1/3 .
Thanks. I know it is frustrating to work on a series only to end up
discarding a big chunk of it. But I think the result here makes a lot
more sense in the long run.
Patches 2 and 3 look OK, but I have a few comments on the first one
(I'll reply separately).
-Peff
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2012-05-11 17:00 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-05-11 13:13 [PATCH 0/3] Reduced re-roll of http-proxy-more branch Nelson Benitez Leon
2012-05-11 17:00 ` Jeff King
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).