Re: Revised Publickey subsystem draft

["Brent McClure" <mcclure%swcp.com@localhost>]

> 3.1 Version Packet
> 
>    Both sides MUST start by sending a version packet that indicates the
>    version of the protocol they are using.
> 
>    string "version"
>    uint32 protocol-version-number
> 
>    The version of the protocol described by this document is version 1.
> 
>    Both sides send the highest version that they implement. The lower of
>    the version numbers is the version of the protocol to use.  If either
>    side can't support the lower version, it should close the subsystem
>    and notify the other side by sending an SSH_MSG_CHANNEL_CLOSE
>    message.
> 
>    Both sides MUST wait to receive this version before continuing.

I think we need to bump the version to 2.

I also just remembered something that I found awkward about the current protocol:

If there is a version mismatch then the channel gets slammed shut with no
opportunity to communicate what the problem was to the other side. It would
be nice to allow for shoving a status packet down the channel indicating
a version mismatch before sending the SSH_MSG_CHANNEL_CLOSE. This would
be a benefit both to clients trying to figure out why their channel won't
open or to admins reading through a server log.

--Brent