search

subdavis / Tusk

🐘 🔒 KeePass-compatible browser extension for filling passwords.
https://subdavis.com/Tusk
Other
482 stars 74 forks source link

[Suggestion] SFTP support #172

Closed conradludgate closed 5 years ago

conradludgate commented 6 years ago

This issue is a

Please describe the current behavior, and explain why it's bad.

Keepass databases cannot be accessed directly via SFTP.

Please describe how you think it should change.

Add support for SFTP. The npm package ssh2 seems to be popular and supports SFTP.

Anything else?

It's supported by Keepass2Android and in the Keepass desktop client through SftpSync, it would be nice to have it on Tusk too.

subdavis commented 6 years ago

Sure, that seems like a reasonable request.

subdavis commented 5 years ago

Cc @graille

Wait a second. SFTP is a totally different protocol than HTTP.

Browsers have no support for native socket connections. The library you cited requires the socket API that only exists in the Node.js runtime.

This feature cannot, as far as I am aware, possibly exist. It's the same reason you can't open an SSH session from a browser.

Closing as wontfix.