bordaigorl / rmview

A live viewer for reMarkable written in PyQt5
GNU General Public License v3.0
755 stars 63 forks source link

rmview keeps asking for passphrase on connect despite having key added to keychain #75

Open siraben opened 3 years ago

siraben commented 3 years ago

rmview keeps asking for SSH passphrase when I run it from the command line, is it not using the keychain?

config.json

{
    "ssh": {
        "address": ["10.11.99.1", "192.168.0.100"],
        "username": "root",
        "key": "~/.ssh/id_rsa",
        "timeout": 2,
        "auth_method": "key"
    },
    "orientation": "portrait",
    "pen_size": 10,
    "pen_color": "red",
    "pen_trail": 200,
    "background_color": "black",
    "hide_pen_on_press": false
}
siraben commented 3 years ago

@bordaigorl ping

bordaigorl commented 3 years ago

Could you try the devel branch? The logic around ssh connections changed a little (although I suspect your issue would still be there). The paramiko configuration bit of the code could do with a bit of rewriting (expert advice welcome). In particular I don't really know how to make it look for system-wide parameters to connect and only on fail ask for them, without trying to connect and reacting on failure. Also: try (on devel) to remove the key setting altogether in the configuration.

siraben commented 3 years ago

@bordaigorl I can still replicate this on devel with the following config.json

{
    "ssh": {
        "address": ["10.11.99.1", "192.168.0.100"],
        "username": "root",
        "key": "~/.ssh/id_rsa",
        "timeout": 2
    },
    "orientation": "portrait",
    "pen_size": 10,
    "pen_color": "red",
    "pen_trail": 200,
    "background_color": "black",
    "hide_pen_on_press": false
}
bordaigorl commented 3 years ago

Is it still a problem with the settings below?

{
    "ssh": {
        "address": ["10.11.99.1", "192.168.0.100"],
        "username": "root",
        "auth_method": "key",
        "timeout": 2
    },
    "orientation": "portrait",
    "pen_size": 10,
    "pen_color": "red",
    "pen_trail": 200,
    "background_color": "black",
    "hide_pen_on_press": false
}
siraben commented 3 years ago

@bordaigorl that fixes it, thanks. The README should probably be updated to reflect this.