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

[der Mouse <mouse%Rodents.Montreal.QC.CA@localhost>]

>> Line 304:
>>    SHOULD be sent.  Note that normally, status messages are only sent by
>> s/only sent by/sent by only/
> I've not done this one - the result struck me as more awkward than
> what we have.  Instead, I've made it "Note that normally, status
> messages are only sent by the server (in response to requests from
> the client)."  I presume this achieves the clarification you were
> after...

It still suffers from the problem: the "only" is attached to the wrong
thing.  See http://webster.commnet.edu/grammar/modifiers.htm for a
short discussion of the issue.  If you really think that this is clear
enough even with "only" in the wrong place, well, go ahead.  I just
point out things I think need fixing.  You don't have to agree with me
about all (or even any) of them; it's your document, not mine. :)

>> 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.
> "For the purposes of a mandatory attribute, mere storage of the
> attribute is not sufficient -- rather, the server must understand and
> implement the intent of the attribute."

Fine by me.

/~\ 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