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

Upgrade to support Erlang/OTP 25 #24

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

Conversation

axelson
Copy link

@axelson axelson commented Jan 2, 2023

Fixes #23

  • Adds list of supported key types, some of which work with a passphrase and some which do not
  • Update readme to open known_hosts with read/write
  • Switch away from deprecated functions

I've tested this with the following key types from ssh-keygen:

  • rsa
  • ed25519
  • ecdsa

I haven't had a successful test with dsa because I wasn't able to get a dsa key to work with my ssh server

My known_hosts file was continually filling up
@pcriv
Copy link

pcriv commented Dec 20, 2023

Hi @axelson are you planning to continue with the upgrade? or do you know of any workarounds for this?

@axelson
Copy link
Author

axelson commented Dec 21, 2023

If I recall correctly this was enough to get it working with Erlang/OTP 25

@axelson
Copy link
Author

axelson commented Aug 14, 2024

Hi, is anyone able to take a look at this?

@tmjoen
Copy link

tmjoen commented Sep 15, 2024

Would be great to get this merged!

@ckoch-cars
Copy link

ckoch-cars commented Dec 16, 2024

@axelson Hello again!

I just found this PR and it helped get us unstuck on a related problem! (using :ssh and reading the ssh key from an env var / app config. )

Specifically we were trying to do 2 things with the erlang :ssh module:
Make a connection and pass in the ssh key from something other than a file on disk
And then with an open connection, bind the connection to a local port (i.e. setup a ssh tunnel)

Thank you (from the future)!

Example (working) code in gist
https://gist.github.com/ckoch-cars/15a88fcb0bc2f4abbed09b49879edf31

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.

Broken on Erlang/OTP 25
4 participants