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