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

Improve support for ssh key files: support files with multiple keys and with already existing options #85

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

Conversation

glebius
Copy link

@glebius glebius commented Feb 25, 2020

Hey, I've send you email 2 weeks ago - no response. Let the pull request hang around now.

has any options.  In this case put comma after end of our options
instead of space.

To tell key with options against bare key we check if key starts
with (ecdsa|ssh)-.  At the moment of this commit OpenSSH supports
the following algorithms “ecdsa-sha2-nistp256”, “ecdsa-sha2-nistp384”,
“ecdsa-sha2-nistp521”, “ssh-ed25519”, “ssh-dss” or “ssh-rsa”.  If
a different algorithm to be introduce to OpenSSH in future, this
code would be broken.  However, adding a full key parser to this
subroutine seems to be overkill now.
@GnUfTw
Copy link

GnUfTw commented Mar 18, 2020

Check out CONTRIBUTING.txt

@glebius
Copy link
Author

glebius commented Mar 18, 2020

Check out [email protected] mailbox, there should be email from me dated 13 Feb 2020

@mkerj
Copy link

mkerj commented Jul 10, 2020

FATAL: R any gitolite-admin git DENIED by fallthru
(or you mis-spelled the reponame)
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

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.

3 participants