Giter VIP home page Giter VIP logo

Comments (6)

abiosoft avatar abiosoft commented on June 1, 2024

It is actually the intended behaviour and there is no problem.

Which is not the indented behavior, as I'm expecting it to pull via SSH. Doing this way apparently fixes the problem:

I noticed Git tries to authenticate even for public repositories when using SSH, so I chose to do the transformation to HTTPS on the fly and use it instead.

May I know why you need SSH for public repository ?
Though I will close this issue because this behaviour does not yield different result for the user and it is intended.

from caddy-git.

Nixtren avatar Nixtren commented on June 1, 2024

I'm actually using a private repository, but I don't need to necessarily specify the SSH key if I have it on ~/.ssh/id_rsa, git seems to assume that by default. That's why I didn't want to specify a key and use the simple one-liner directive.
When people go to clone a GitHub repository, GitHub automatically gives them the HTTPS link by default:
screenshot

For that reason I think the transformation to HTTPS is unneeded. And in my case, it results in unintended behavior :P

from caddy-git.

abiosoft avatar abiosoft commented on June 1, 2024

Bear in mind that it is implemented to be as standalone as possible. Imagine running it in a container or not having ssh keys configured. That is why you need the key specified.

I choose not to believe that additional curly braces and an extra line is that much of deal.

from caddy-git.

Nixtren avatar Nixtren commented on June 1, 2024

Fair enough 👍

I didn't want to specify the key due to another issue, it couldn't find my key at all:

Warning: Identity file .ssh/id_rsa not accessible: No such file or directory.

But that's another issue that I will hopefully fix. Removing the key parameter seemed like a quick option to remove that warning, but I noticed the middleware put "https://" before my Git SSH url, that's why I opened this issue.

from caddy-git.

abiosoft avatar abiosoft commented on June 1, 2024

Specify absolute path for key.

from caddy-git.

Nixtren avatar Nixtren commented on June 1, 2024

That solved it, thanks!

from caddy-git.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.