IETF-SSH archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: I-D ACTION:draft-ietf-secsh-publickey-subsystem-03.txt



> 	Filename	: draft-ietf-secsh-publickey-subsystem-03.txt

This looks good.  I do see a few copy-edit details, whch I shall leave
it up to others to decide on the actual importance of:

Lines 242-244:
   The version packet, as well as all requests and responses described
   in Section 3 are a description of the 'name' field and the data part
   of the packet.
s/3/3,/

Line 304:
   SHOULD be sent.  Note that normally, status messages are only sent by
s/only sent by/sent by only/

Line 367:
   SSH_PUBLICKEY_ACCESS_DENIED
s/$/./

Lines 372-375:
   SSH_PUBLICKEY_ATTRIBUTE_NOT_SUPPORTED.  For the purposes of a
   mandatory attribute, storage of the attribute is not sufficient, but
   requires that the server understand and implement the intent of the
   attribute.
That "requires" looks subjectless; strictly, it appears to refer back
to "storage" eight words earlier, but that reads nonsensically.  While
I think the meaning is clear, I'm not confident non-anglophones will
find it as clear as I do; you might want to reword it.  Suggestion:
   SSH_PUBLICKEY_ATTRIBUTE_NOT_SUPPORTED.  For the purposes of a
   mandatory attribute, mere storage of the attribute is not
   sufficient; the server must understand and implement the intent of
   the attribute.

/~\ The ASCII				der Mouse
\ / Ribbon Campaign
 X  Against HTML	       mouse%rodents.montreal.qc.ca@localhost
/ \ Email!	     7D C8 61 52 5D E7 2D 39  4E F1 31 3E E8 B3 27 4B



Home | Main Index | Thread Index | Old Index