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

Add ability to push to server over ssh with public/private key #218

Closed
geoffreyvanwyk opened this issue Dec 17, 2013 · 4 comments
Closed

Comments

@geoffreyvanwyk
Copy link

No description provided.

@davvid
Copy link
Member

davvid commented Jun 4, 2014

Hello.. I think this is similar to #262 .. do the links mentioned there help at all? This topic has come up a few times already. What OS are you on?

It might be worth documenting this topic in a FAQ section so that others don't have to keep rediscovering the answer. I hope that helps.

@davvid davvid closed this as completed Jun 4, 2014
@Vdragon
Copy link
Contributor

Vdragon commented Oct 7, 2014

@systemovich Hi, please checkout the updated issue #262 if it's your issue.

davvid added a commit to davvid/git-cola that referenced this issue Apr 26, 2015
"git push" over ssh will not use SSH_ASKPASS unless we are fully
detached from the terminal.  It is not enough to redirect stdin
and set the SSH_ASKPASS environment variable ~ we need to call
os.setsid() after forking in the child process.

Use subprocess.Popen's preexec_fn to call os.setsid() when
executing git commands.  This ensures that ssh will always use
the {GIT,SSH}_ASKPASS programs, even when git-cola is launched
from the terminal.

The end result is that Git's password prompts will use the
configured askpass program rather than prompting in the
terminal.

Closes git-cola#218
Closes git-cola#262
Closes git-cola#377
Reported-by: V字龍(Vdragon) <Vdragon.Taiwan@gmail.com>
Reported-by: Geoffrey van Wyk <geoffrey@vanwyk.biz>
Helped-by: Daniel Harding <dharding@living180.net>
Helped-by: Pamela Strucker <struckerpamela@gmail.com>
Helped-by: Luke Bakken <luke@bakken.io>
Helped-by: Stanisław Halik <sthalik@misaki.pl>
Helped-by: JiCiT <jason@infinitebubble.com>
Signed-off-by: David Aguilar <davvid@gmail.com>
@davvid
Copy link
Member

davvid commented Apr 26, 2015

Please see #377 for the solution. cheers

@geoffreyvanwyk
Copy link
Author

@davvid It is working now, thanks.

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

No branches or pull requests

3 participants