Thomas Gerbet (tgerbet)2021-01-08 14:20 Declining as this is not expected to change. Status changed from Waiting for information to Declined
Thomas Gerbet (tgerbet)2020-11-16 11:15 @lorentzr @goyotm Following internal discussions it looks like can be closed/marked as WONTFIX as it looks like there is nothing to fix here?
Thomas Gerbet (tgerbet)2020-11-05 17:47 Okay, it would have been nice to have the repro case in the description of the request but it was not obvious we were specifically talking about the git-lfs-authenticate SSH command. My first question still remains: how users ended up using such URLs? I'm a bit reluctant to introduces changes increasing the complexity in a sensitive area to accommodate something that is not a public behavior and that is not even tested somewhere in the codebase (as far as I can see).
Martin GOYOT (goyotm)2020-11-05 17:29 last edited by: Martin GOYOT (goyotm) 2020-11-05 17:31 We are talking about this (plus my previous comment about the validity of different use cases): ~/t/git ❯❯❯ ssh gitolite@dev4mjagger git-lfs-authenticate use-me/refia.git download {"href":"https:\/\/tuleap-web.tuleap-aio-dev.docker\/plugins\/git\/use-me\/refia.git\/info\/lfs","expires_in":600,"header":{"Authorization":"RemoteAuth 1.874ee5e63a617d80957d4e0a46aaf8f98caef0fe6611ba104454a4d1ae17fe11"}} ~/t/git ❯❯❯ ssh gitolite@dev4mjagger git-lfs-authenticate use-me/refia download git-lfs-authenticate arg 1 must be a valid repository
Thomas Gerbet (tgerbet)2020-11-05 17:22 What are we talking about here? What's a "classic repository"? AFAIK there is no good reason for the /{project_name}/{path:.*\.git}/info/lfs* endpoints to accept paths without .git. What issue are we trying to fix here? The way Git LFS guesses the server works even if the the remote URL does not end with .git [0]. [0] https://github.com/git-lfs/git-lfs/blob/v2.12.1/docs/api/server-discovery.md#guessing-the-server
Martin GOYOT (goyotm)2020-11-05 17:18 last edited by: Martin GOYOT (goyotm) 2020-11-05 17:19 As of today: cloning with HTTP: - with .git with or without LFS ✅ - without .git with or without LFS ✅ cloning with SSH: - with .git with or without LFS ✅ - without .git without LFS ✅ - without .git with LFS 🛑 I agree none of the URL does not contain the .git at the end. But for the sake of consistency, I would advocate either it works or it doesn't but not that it works in almost all cases but one.
Lorentz Romain (lorentzr)2020-11-05 17:12 I agree with the idea but I think it's inconsistent with the classic repositories.
Thomas Gerbet (tgerbet)2020-11-05 16:53 Hi, Why should it work? Was there a moment in when the URLs exposed by Tuleap to clone a repository did not contain .git at the end? Status changed from Under implementation to Waiting for information