From: Christoph Blank <sol@textmode.at>
To: Git List <git@vger.kernel.org>
Subject: Re: Http/s authentication failure with some git versions
Date: Mon, 25 Jul 2011 21:39:51 +0200 [thread overview]
Message-ID: <20110725193951.GB20953@tp> (raw)
In-Reply-To: <20110725163223.GB7071@sigill.intra.peff.net>
Thanks a lot for the pointer!
I will upgrade to a newer version
greetings,
Christoph
On Mon, Jul 25, 2011 at 10:32:23AM -0600, Jeff King wrote:
> On Mon, Jul 25, 2011 at 03:23:17PM +0200, Christoph Blank wrote:
>
> > Following command:
> > git clone "https://git%40app.example.com:myp...@git.example.com/app-comm
> >
> > This works fine with git versions:
> > 1.7.4.1
> > 1.7.6
> >
> > and fails with:
> > 1.7.1
> > 1.7.2.5
> > 1.7.33
> >
> > does anyone know why, or if there was a particular bug fixed?
> > I don't want to upgrade git on the production systems without a reason..
> > Can't find anything and I'm getting desperate...
>
> I skimmed through:
>
> git log v1.7.3.3..v1.7.4.1 http.c
>
> and came up with:
>
> commit f39f72d8cf03b61407f64460eba3357ec532280e
> Author: Gabriel Corona <gabriel.corona@enst-bretagne.fr>
> Date: Sun Nov 14 02:51:15 2010 +0100
>
> Fix username and password extraction from HTTP URLs
>
> Change the authentification initialisation to percent-decode username
> and password for HTTP URLs.
>
> which seems a likely candidate based on the example you gave. You could
> try cherry-picking that commit onto your older version if you don't want
> to do a full upgrade.
>
> Now, I happened to know that http.c was where the relevant code lived
> (because I was working on it recently), and limiting my search to
> commits that touched that file made the list quite short. But knowing
> nothing else except a test case that works on some versions and not on
> others, you could always "git bisect", which would find the relevant
> commit for you.
>
> -Peff
> --
> To unsubscribe from this list: send the line "unsubscribe git" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
prev parent reply other threads:[~2011-07-25 19:40 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-07-25 13:23 Http/s authentication failure with some git versions Christoph Blank
2011-07-25 16:32 ` Jeff King
2011-07-25 19:39 ` Christoph Blank [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20110725193951.GB20953@tp \
--to=sol@textmode.at \
--cc=git@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.