ssh_authorize duplicates keys

[eykd]

I would expect ssh_authorize to be idempotent, but running it repeatedly results in duplicate keys in .ssh/authorized_keys.

I attempted to rewrite it using something like file_update(keyf, lambda _: text_ensure_line(_, key)) but this produces duplicate lines as well.

[superbobry]

Hm, this is weird -- file_update looks perfectly fine to me, except for this EOL thing.

[sebastien]

It might be time to write some test cases -- will investigate that, thanks!

[superbobry]

Good idea! it's always the time for tests :)

[eykd]

Tests would be great. :) I agree it looks right, but it results in duplicate lines. :/ I'm going to see if I can fix it in my fork--I've got some other fixes that I'd like to send in a pull request eventually.

[sebastien]

That would be amazing, thanks.

[sebastien]

Fixed in 40f1fc8