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.

Sign up for GitHub

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

Jump to bottom

SSH Authorization #8

Merged
merged 5 commits into from
Sep 12, 2014
Merged

SSH Authorization #8

merged 5 commits into from
Sep 12, 2014

Conversation

unusualbob
Copy link
Collaborator

Basic building blocks for ssh authentication/authorization, still early alpha but with this I can at least:

ssh -T git@localhost
Hello unusualbob! You have successfully authenticated with shipright.

unusualbob and others added 2 commits September 8, 2014 00:37
@unusualbob
initial untested ssh key stuff
f941c97
@unusualbob
Functioning -T ssh authentication
31f7a6d 
requires a designated user and chmod +x for that user on git-shell
keyManager could use work on users who have existing authorized_keys file
@martindale
Copy link
Owner

Can I merge this now and take it over, or do you still want to do more work?

@unusualbob
Copy link
Collaborator Author

I made a little bit of progress today, got to the point where git client
actually barely talks to the server it just gets stuck for some reason.

I can push that when I get back to my computer as another branch or
something though if you like.
On Sep 9, 2014 8:07 PM, "Eric Martindale" notifications@github.com wrote:

Can I merge this now and take it over, or do you still want to do more
work?

—
Reply to this email directly or view it on GitHub
#8 (comment).

@unusualbob
Copy link
Collaborator Author

I managed to get this to work now :D

@unusualbob
Fully documented gitshell, slight refactor. Added required npm module…
3a2b706 
…s to package.json, cleaned up keyManager and included fixes for paths with spaces, keymanager now correctly adds keys to authorized_keys
@unusualbob
get git working without needing kexec
378e5a3
@unusualbob
Copy link
Collaborator Author

Just managed to figure out what was going on with streams and now this no longer requires kexec. I'm thinking the best way to notify the server about commit events is probably just a post-receive hook for each repo.

@unusualbob unusualbob changed the title WIP Feature - ssh authorization SSH Authorization Sep 11, 2014
martindale added a commit that referenced this pull request Sep 12, 2014
@martindale
Merge pull request #8 from unusualbob/feature/ssh_authorization
bca4892 
SSH Authorization
@martindale martindale merged commit bca4892 into martindale:develop Sep 12, 2014
@martindale martindale mentioned this pull request Sep 12, 2014
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@unusualbob @martindale