-
Notifications
You must be signed in to change notification settings - Fork 937
feat: wrap ssh with coder key #894
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
Conversation
Codecov Report
@@ Coverage Diff @@
## main #894 +/- ##
==========================================
- Coverage 65.75% 65.70% -0.06%
==========================================
Files 216 215 -1
Lines 13834 13725 -109
Branches 103 103
==========================================
- Hits 9097 9018 -79
+ Misses 3807 3785 -22
+ Partials 930 922 -8
Continue to review full report at Codecov.
|
You'll have to pipe the SSH command to |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We'll need to abstract agent authentication out of workspaceagent.go
, and I think that's the only real blocker here. Other than that, I just had a buncha lil UX knobs to twist.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems like we agree on all the comments, so I shall approve! Great work on this- 'tis a top-tier feature (if you'd still like my thoughts just tag again).
@deansheather top-tier review 🏹🏹🏹 |
What this does
GIT_SSH_COMMAND
environment variable tocoder gitssh --
. The double dash allows git to pass the command flags and they be ignored by cobra.coder gitssh
will grab the user's private key fromcoderd
, write it to a temp file, and runssh -i /tmp/my-coder-key
.coder publickey
command so users can upload this key to their git provider.