IETF-SSH archive

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

WG Chair Copyediting for draft-ietf-secsh-break-02



I found a few nits:

1) I Garbled sentence in the
security considerations section:

   Alternatively, support for the BREAK facility MAY be imlemented
   configurable or a per port or per server basis.

The meaning is clear but could definitely use some sanding and
polishing.

2) misplaced comma and missing article:

change:

   If  BREAK duration request of less than 500ms, is
   requested a BREAK of 500ms SHOULD be sent since most devices will
   recognize a BREAK of that length.

to:

   If a BREAK duration request of less than 500ms is requested, a BREAK
   of 500ms SHOULD be sent since most devices will recognize a BREAK
   of that length.

3) spelling error, ambiguity:

   If
   a BREAK of any kind was preformed, SSH_MSG_CHANNEL_SUCCESS MUST be
   sent.  If no BREAK was preformed, SSH_MSG_CHANNEL_FAILURE MUST be

s/preformed/performed or passed on/

(fix typo; clarify that the server doing break-passthrough NEED NOT
wait for the break to be ack'ed, as this may not be possible to do
reliably in some cases).

---

And, I couldn't help myself:

Break-length section rewording:

   A BREAK-length parameter of 0 is a request for a default-length break;
   by default, this SHOULD be a 500ms break.

   Some implementations will not be in a position to control the
   length of a BREAK; such implementations SHOULD instead generate an
   fixed-length break of an implementation-defined length in response
   to this request.

   Others SHOULD limit the minimum and maximum length of BREAK
   durations.  The encoding specified above allows a sender to request
   a maximum BREAK duration of approximately 49.7 days.  

   By default, durations less than 500ms SHOULD be silently extended
   to 500ms (as some devices may not recognize BREAKS much shorter
   than this); durations longer than 3000ms SHOULD be silently truncated to
   3000ms.

   Implementations MAY allow an administrator to adjust the default, minimum,
   and maximum break lengths.

---

Let's have a quick 1-week WG review on this followed by a respin.

WG members: please review this document and send comments on or before
4/28/2004

						- Bill







Home | Main Index | Thread Index | Old Index