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

Adding some ways of asking for a password, aka SSH support #803

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

shana
Copy link
Member

@shana shana commented May 29, 2018

Fixes #244

SSH_ASKPASS/GIT_ASKPASS will ask for a password if there isn't one. If the command
line asks for a password, use a custom UI to ask for it. I haven't tested the custom UI at all because haha I can't get it to trigger, processes that execute with redirected input don't ask for things on the command line. But for windows, at least, SSH_ASKPASS will ask for a password (though it won't cach it apparently).

This also moves reading process output to a thread because if the process asks for input without us noticing, it will hang forever and hang Unity in the process when it wants to kill the domain. This checks if we need to shutdown every second and aborts if needed.

…work operations

SSH_ASKPASS/GIT_ASKPASS will ask for a password if there isn't one. If the command
line asks for a password, use a custom UI to ask for it.
@shana shana force-pushed the enhancements/ssh-support branch from edb884f to 25e068a Compare May 29, 2018 13:03
# Conflicts:
#	src/GitHub.Api/Application/ApplicationManagerBase.cs
@shana shana changed the title Adding some ways of asking for a password Adding some ways of asking for a password, aka SSH support Jun 1, 2018
@ghost
Copy link

ghost commented Nov 16, 2018

I had to change my environment variable to make SSH askpass working on windows 10

@ibytergj
Copy link

ibytergj commented Sep 9, 2019

Hi what is the status of using ssh for GitHub for Unity?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

SSH support
3 participants