-
-
Notifications
You must be signed in to change notification settings - Fork 102
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
[release] NPE during tagging #1435
Comments
Some more context on this one: my git remote was configured to use SSH, not HTTPS. This seems to generally be unsupported with JReleaser currently, since even when I added that dependency, things wouldn't work. It'd still try to authenticate with user name and PAT (as the IMO, the best solution to all this would be to use the native git installation (by launching an external process) instead of using jgit. Then I should be able to use key-based auth, with the key passphrase being retrieved from my SSH agent. Another issue with jgit is that the jsch dependency isn't actively maintained, so newer key formats like ed25519 aren't supported. I got unstuck by changing my remote to use HTTPS, I think it's worth calling out in the docs that SSH isn't supported atm.? And/or fail eagerly with a descriptive error message when detecting that SSH is used. |
Failing quickly when SSH is configured as protocol sounds like a quick win we should support right away. Switching to external git command will take more effort but it's feasible. |
Edge case here - only realized the problem was SSH after I found this issue. I had git configured to globally use ssh instead of https, which I feel is fairly common. It would be a good idea to call this out in the docs, or more specifically, the checklist. Thanks! |
FWIW regarding the state of JGit SSH support via JSch This bundle provides an implementation of git transport over SSH implemented via JSch. This bundle should be considered deprecated. It is essentially unmaintained, and the JGit project may decide anytime to remove it completely without further ado. The officially supported SSH transport is in bundle org.eclipse.jgit.ssh.apache and is built upon Apache MINA sshd. |
Getting this one:
Seems like that
org.eclipse.jgit.ssh.jsch
dependency would need to be added?The text was updated successfully, but these errors were encountered: