Re: wishlist; unify behavior while cloning non-bare repos over http to be in line with ssh/local
- From
Jacob Keller <jacob.keller@gmail.com>
- Date
- May 10, 2016, 20:33 UTC
- Message-ID
- <CA+P7+xoCmMV01=11GRVsuOSoTPgzSFT-VAeqa1SCcLnzpGcb0Q@mail.gmail.com>
- In-Reply-To
- <xmqqfutpoepz.fsf@gitster.mtv.corp.google.com>
On Tue, May 10, 2016 at 1:11 PM, Junio C Hamano <gitster@pobox.com> wrote:
Show 26 quoted lines
> Yaroslav Halchenko <yoh@onerussian.com> writes: > >> On Fri, 06 May 2016, Yaroslav Halchenko wrote: >> >>> Dear Git Folks, >> >>> Originally this issue was mentioned in previous thread [1], and I have decided >>> to bring it into a separate thread. ATM there is a dichotomy in git behavior >>> between cloning non-bare repos: if I clone over ssh or just locally by >>> providing url without trailing /.git, git senses for /.git and works just fine >>> with ssh or local repositories, but fails for "dummy" http ones, the demo >>> script is here [2] which produces output listed below. From which you can see >>> that cloning using http URL to the repository without /.git fails (git version >>> 2.8.1, Debian). As it was noted in [1], concern could have been to not >>> traverse website since could lead to dangerous places. But .git is under >>> originating url directory, as well as info/ or HEAD or any other object >>> accessed by git, so IMHO this concern is not a concern. > > I am afraid that the reason why you saw no response is primarily > because nobody is interested in extending dumb commit-walker HTTP > transport after the world has largely moved on and abandoned it. > > The necessary update to the client might as simple as using > $GIVEN_URL/.git/ and attempting the request again after seeing the > probe for $GIVEN_URL/info/refs fails. > --
I know at least Jenkin's Git plugin has a workaround to solve this issue that is quite similar.
Thanks, Jake