Re: [PATCH] remote-curl: don't fall back to Basic auth if we haven't tried Negotiate

From: Dmitry Vilkov <dmitry.a.vilkov@gmail.com>
Date: Feb 25, 2016, 16:54 UTC
Message-ID: <CAHdYDCqM0dhjs0vYmuHmbU3Pr=Wmfp_7GbgXGsRcbT+TgX10vA@mail.gmail.com>
In-Reply-To: <xmqqk2lz9i66.fsf@gitster.mtv.corp.google.com>

Hi, guys!

> I sent in a patch (and I believe I CC'd you) that adds an option
> http.emptyAuth that can be used in this case.  It should make its way to
> a future release.

Somehow I've missed your letter...

Show 6 quoted lines

> The patch has been queued as 121061f6 (http: add option to try
> authentication without username, 2016-02-15); perhaps you can help
> by trying it out in your installation before it hits a released
> version of Git?  That way, if the patch does not fix your problem,
> or it introduces an unexpected side effect, we would notice before
> we include it in a future release.

I've cherry-picked commit 121061f6 over version 2.4.10 and 2.7.1. In both cases it works exactly as expected.

Please, let me know if I can help with something else regarding this issue.

2016-02-21 0:38 GMT+03:00 Junio C Hamano <gitster@pobox.com>:

Show 19 quoted lines

> Dmitry Vilkov <dmitry.a.vilkov@gmail.com> writes:
>
>> Hi guys! Any luck with fixing this issue?
>
> I think Brian suggested an alternative approach, to which you earler
> responded
>
>>> That would be great! Definitely it will be much better solution than
>>> patch I've proposed.
>
> The patch has been queued as 121061f6 (http: add option to try
> authentication without username, 2016-02-15); perhaps you can help
> by trying it out in your installation before it hits a released
> version of Git?  That way, if the patch does not fix your problem,
> or it introduces an unexpected side effect, we would notice before
> we include it in a future release.
>
> Thanks.
>