git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [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).