Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

git commands with an sso origin fail with Error: invalid argument: 'port'; class=Invalid (3) #4177

Open
antholeole opened this issue Jul 29, 2024 · 2 comments
Labels
🐛bug Something isn't working

Comments

@antholeole
Copy link

Description

I have an sso url as a git origin: sso://some-org/some-project. running any jj git command fails with Error: invalid argument: 'port'; class=Invalid (3). jj git fetch, jj git push -b my-branch, etc. It even fails to clone; jj git clone <sso url> fails with the same error.

In practice, I'm attempting to use jj in a google internal gerrit instance.

Steps to Reproduce the Problem

run jj git clone sso://anything which will fail before it attempts to make the api call. I don't think you actually need a sso repository setup, but if a googler looks at this ticket I can provide them my internal gerrit instance url to test it on.

Expected Behavior

it clones the repo

Actual Behavior

it fails with invalid argument port

Specifications

  • Platform: linux
  • Version: 0.17.1
@jonathantanmy
Copy link
Collaborator

(I also work for Google.)

URLs of the form sso://... (or anything that is not what you'd expect to be generally supported) work in the Git CLI due to an extension mechanism called remote helpers (https://git-scm.com/docs/gitremote-helpers), but the Git library that jj uses to fetch, libgit2, does not seem to support those. You can reach out internally if you want to discuss options.

@ilyagr
Copy link
Collaborator

ilyagr commented Jul 30, 2024

In principle, we could consider shelling out to git push from jj git push to do the actual pushing, similarly to #2920. This would solve other incompatibilities as well. It would be annoying to maintain, though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants