-
-
Notifications
You must be signed in to change notification settings - Fork 35
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
Github ssh connection issues #9
Comments
Thanks for the report @Folcon . Do you see any log messages about fetching the ssh keys? Example of the logs:
If that's fine, it could be the type of the public keys that aren't accepted by the Upterm ssh server. What type of ssh key do you use? Is it RSA or ed25519? I believe legacy DSA keys might not be supported. |
I see it's ed25519, https://github.com/Folcon.keys . Can you try adding a RSA key temporarily for testing? |
Nope:
Ok, it worked this time. Not sure if it's an intermittent issue, but it does work at the moment. |
I'll close the issue if it's resolved. It could also be a upterm server issue. It's possible to run your own upterm server on Heroku. I haven't tried that myself yet.
btw. that doesn't seem to work at all and the connection will always fail in that case. |
You might want to add an example connection string in the docs so it's
clearer about what it should look like.
- Nilu
…On Wed, 2 Feb 2022, 08:43 Lari Hotari, ***@***.***> wrote:
Ok, it worked this time. Not sure if it's an intermittent issue, but it
does work at the moment.
I'll close the issue if it's resolved. It could also be a upterm server
issue. It's possible to run your own upterm server on Heroku. I haven't
tried that myself yet.
I also tried stripping the password string, as the connection string
instruction in the docs is a little ambiguous.
btw. that doesn't seem to work at all and the connection will always fail
in that case.
—
Reply to this email directly, view it on GitHub
<#9 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAA6BL5MOUMWOKTQWUN276DUZDVB7ANCNFSM5NBJL5WA>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Same issue here - key seems to be accepted, but then connection is closed:
No error or anything else - just a dead session. |
@jas88 Does the connection get established when you don't limit access to specific github users? |
@lhotari Yes - I removed the user list as a test, and it worked fine then. I've fixed the immediate issue I needed (turned out to be a regression in .Net 6.0.1's Kerberos handling, which was making our Github CI test hosts crash) - which was made much easier by having this action available, thank you. |
Can confirm this issue is still occurring on ssh -i ~/.ssh/gh_rsa [email protected]
Connection closed by 157.230.199.75 port 22 |
Commenting here since this can help I was trying this github action & was facing Followed below steps , generating ed25519 key & using it for ssh to uptermd.upterm.dev worked fine for me.
|
There's also some notes about RSA keys in the README of upterm. |
One of the commits removed the instructions for the workarounds: owenthereal/upterm@0f410ca |
Same issue. I'm not limiting connections to any actor. |
I solved it by adding an ssh key to Github. |
Hey,
I'm not sure if the docs are unclear or something is wrong, but I couldn't use either
limit-access-to-actor
together withlimit-access-to-users
orlimit-access-to-actor
by itself (introduced in #8).The ssh command I used was:
I also tried stripping the password string, as the
connection string
instruction in the docs is a little ambiguous.To sanity check I tried doing an ssh connection test to github, which worked fine:
Not sure what the problem is or if this is an upstream issue...
The text was updated successfully, but these errors were encountered: