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

Update connecting-to-ev3dev-with-ssh.md #445

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tim-band
Copy link

@tim-band tim-band commented Nov 3, 2024

-o PreferredAuthentications=password is necessary if your ssh client is set to try other methods first. We could instead offer this as a response to the error message "Received disconnect from 192.168.0.1 port 22:2: Too many authentication failures"

-o PreferredAuthentications=password is necessary if your ssh client is set to try other methods first.
We could instead offer this as a response to the error message "Received disconnect from 192.168.0.1 port 22:2: Too many authentication failures"
@dlech
Copy link
Member

dlech commented Nov 3, 2024

This seems like something that should be set in a ssh config file rather than specifying it on the command line every time.

@tim-band
Copy link
Author

tim-band commented Nov 9, 2024

This seems like something that should be set in a ssh config file rather than specifying it on the command line every time.

Yes I agree, but saying "and now add this stanza to your .ssh/config" file seems way too much, saying "if you get 'too many auth failures' add this to your .ssh/config" is even more... people who know about .ssh/config will know to do it, people who don't won't want to learn it here.

Saying "if you get 'Too many authentication failures' try adding '-o PreferredAuthentication=password" seems maybe about right, but just adding it is saying even less, which is what I suggest here, but either of these seems OK to me.

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.

2 participants