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

Check SSH_AUTH_SOCK to determine existence of ssh-agent #239

Merged
merged 3 commits into from
Apr 26, 2022

Conversation

talpr
Copy link
Contributor

@talpr talpr commented Apr 25, 2022

See discussion in #81.

@martinvonz
Copy link
Member

Thanks for looking into this. I had just used SSH_AGENT_PID without thinking about it before.

Could you add mention of it in the changelog? In particular, it may be useful to say something about who may have run into the problem before. I'm hoping it may help convince some users who have tried jj before and just gave up because authentication didn't work for them. Perhaps something like this:

  • ssh-agent is now detected even if $SSH_AGENT_PID is not set (as long as $SSH_AUTH_SOCK is set). This should help at least macOS users where ssh-agent is launched by default and only $SSH_AUTH_SOCK is set.

Actually, is that the case for you? That you're on macOS and have not manually started ssh-agent, I mean?

@talpr talpr force-pushed the talp/ssh-auth-sock branch from 0cd65c2 to 014ed24 Compare April 26, 2022 08:38
@talpr
Copy link
Contributor Author

talpr commented Apr 26, 2022

Done.

Actually, is that the case for you? That you're on macOS and have not manually started ssh-agent, I mean?

Yup, basically. TBH, I don't remember ever starting the agent myself, not on Macs and not on Linux machines, but I have no idea what magic might have been running it for me all this time. 😅

@talpr talpr force-pushed the talp/ssh-auth-sock branch from 014ed24 to e3b6af5 Compare April 26, 2022 08:54
@martinvonz martinvonz merged commit 4a1576c into jj-vcs:main Apr 26, 2022
@talpr talpr deleted the talp/ssh-auth-sock branch April 27, 2022 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants