git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* git+ssh protocol regression
@ 2010-06-22  9:22 Pascal Obry
  0 siblings, 0 replies; only message in thread
From: Pascal Obry @ 2010-06-22  9:22 UTC (permalink / raw)
  To: Git Mailing List

With current git version doing:

   $ git clone git+ssh://server/path
   fatal: I don't handle protocol 'git ssh'

The '+' has been changed to ' '. This looks like the protocol in the URL
has been decoded (a + is an encoded space).

The culprit is:

<<
commit 9d2e942070b235e1f4a0b8dd4c55a6b3f0fe914a
Author: Jeff King <peff@peff.net>
Date:   Sun May 23 05:19:44 2010 -0400

    decode file:// and ssh:// URLs

    We generally treat these as equivalent to "/path/to/repo"
    and "host:path_to_repo" respectively. However, they are URLs
    and as such may be percent-encoded. The current code simply
    uses them as-is without any decoding.

    With this patch, we will now percent-decode any file:// or
    ssh:// url (or ssh+git, git+ssh, etc) at the transport
    layer. We continue to treat plain paths and "host:path"
    syntax literally.

    Signed-off-by: Jeff King <peff@peff.net>
    Signed-off-by: Junio C Hamano <gitster@pobox.com>
>>

I've not seen discussion about this regression. I don't think it is intended.

Patch will be sent in a followup message.

Pascal.

-- 

--|------------------------------------------------------
--| Pascal Obry                           Team-Ada Member
--| 45, rue Gabriel Peri - 78114 Magny Les Hameaux FRANCE
--|------------------------------------------------------
--|              http://www.obry.net
--| "The best way to travel is by means of imagination"
--|
--| gpg --keyserver wwwkeys.pgp.net --recv-key C1082595

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2010-06-22  9:22 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-06-22  9:22 git+ssh protocol regression Pascal Obry

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).