cURL / Mailing Lists / curl-library / Single Mail

curl-library

Re: patch to allow for ssh md5 checking option

From: Daniel Stenberg <daniel_at_haxx.se>
Date: Mon, 24 Sep 2007 23:01:44 +0200 (CEST)

On Mon, 24 Sep 2007, Johnny Luong wrote:

(Please don't top-post)

> Yah, I was thinking of that too, but I needed something now (for internal
> use) and the known_hosts format that openssh uses is somewhat complicated.

I want to second Dan's comment here that the libcurl part was perfect, but the
curl tool end is not really that convenient. How would a user even get the
md5sum for the public key? With an option like this, won't we also need an
option that can store the key this way for use at later invokes?

> My hope was maybe it would go in so I don't maintain a separate branch

I like the patch and I think we should work on getting it added!

> at some later point, another option would be provided which would use a
> local database (e.g: known_hosts).

I think it would make the most sense to support the openssh format of that
file then. Don't you?

Oh, and a little nit about the code style in the patch... There were multiple
lines longer than 80 columns:

   http://curl.haxx.se/docs/contribute.html#Line_Lengths

-- 
  Commercial curl and libcurl Technical Support: http://haxx.se/curl.html
Received on 2007-09-24