-
-
Notifications
You must be signed in to change notification settings - Fork 4
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
Support SSH key auth #15
Comments
Thanks for the request. I'll add it to my list of improvements to look at. |
Also interested in using SSH keys rather than password. https://www.youtube.com/watch?v=u2iQACDuoeU if its useful |
@soricesg @JitteryDoodle I'm currently experimenting with this feature and will let you know if I get this working |
@JitteryDoodle @soricesg I managed to get it working. Was actually easier than I thought it would be to implement since I am already using pythons ssh library. Had to change the config flow from single step to multi-step flow that now looks like this. First Step: Enter Unraid Hostname / IP, username and select authentication method Second Step: Specify location on HA were key is kept Last Step: Specify port, UPS and sensor update settings And the integration is up and running. I'll add this as a pre-release feature soon and you can try it out |
@soricesg @JitteryDoodle When you have time feel free to try out the multi-step-auth feature.
I provided a configuration guide here but I'm sure you know how ssh keys work 😃 https://github.com/domalab/ha-unraid/wiki/Unraid-Integration-Setup-Guide |
I’m away from my devices for the next week or 2 and don’t want to risk doing anything remotely while I’m away. I’ll absolutely give it a try and any feedback once I’m back home |
Hi @domalab, wanted to try ssh key auth as well. Followed your steps for manual install and got a error during adding the integration: Home assistant log: const.py seems to be missing from the commit after adding these consts with some own random keys the integration works 👍 unraid.execute_command seems to fail with the following message:
Unraid (please complete the following information):
Home Assistant (please complete the following information):
|
Thanks for picking up the missing entries in const.py. I re-synced the branch and its now back. |
@l3ztum Regarding the action: unraid.execute_command failed. Can you please follow this updated wiki article? https://github.com/domalab/ha-unraid/wiki/How-to-Use-Unraid-Service-Commands My fault, I forgot to update the article that IP addresses are not supported anymore. |
Is your feature request related to a problem? Please describe.
My server only uses SSH keys and password auth is disabled. Because of this, it's not possible to complete the integration setup.
Describe the solution you'd like
It would be great if SSH key support was added.
Use Case or Why Do You Want This?
Describe a specific scenario where this feature would be helpful or explain why you think it's necessary.
Remove reliance on SSH username and password
Priority of Request
How important is this feature to you?
Additional context
Add any other context or screenshots about the feature request here.
The text was updated successfully, but these errors were encountered: