Re: AD review of msgtrk documents

Tony Hansen <[email protected]> Wed, 21 Aug 2002 16:33:57 -0400
Newsgroups gmane.ietf.msgtrk
Message-ID <[email protected]>
Thank you Ned. I have a few comments below, plus some questions for the 
group.

	Tony

[email protected] wrote:
> Overall these look good. Almost all of the changes are purely editorial in
> nature.
> ...
> 
> mqtp-08
> 
> The copyright date is 1999. This seems wrong...

Obviously the documents have been in the hopper for too long. :-)




> Section 2.4. Should say something about client timeouts and how
> long it is appropriate to wait for a server.

I'd like to propose the following additions:

* In section 2.5 "Firewall Considerations", where it talks about the 
server doing chaining operations, add the following:

	If a server chooses to perform a chaining operation itself, it MUST 
provide a response within 2 minutes, and SHOULD return a "no further 
information is available" response if it cannot provide an answer at the 
  end of that time limit.

* In section 2.4 Optional Timers, add:

	An MTQP client MAY have an inactivity autologout timer while waiting for 
a response from the server. Since an MTQP server may be a firewall, and 
may be chaining information from other servers, such a timer MUST be at 
least 2 minutes in duration.

* I arbitrarily chose 2 minutes. If there's a better time limit, feel 
free to suggest one.

* I also plan on swapping sections 2.4 and 2.5, since Optional Timers 
now refers to firewall support.




> Section 4. It seems appropriate to have two qualified error
> responses to TRACK: (1) An indication that TLS must be
> negotiated before this message can be tracked and (2) An indication
> that the search succeeded but found no result. 

I'd like to propose the following additions to section 4:

	A negative response to the TRACK command may include these reason codes:

	"/" "tls-required"
	"/" "admin"
	"/" "unavailable"

	The reason code "/tls-required" SHOULD be used when the server has 
decided to require TLS. The reason code "/admin" SHOULD be used when the 
server has become unavailable, due to administrative reasons, since the 
connection was initialized. The reason code "/unavailable" SHOULD be 
used when the server has become unavailable, for other reasons, since 
the connection was initialized.



> The URL registration in section 9 doesn't seem to meet the
> requirements set forth in RFC 2717. In particular, the
> URL registration template needs to be included.

I'll review this.



> Section 10. The IANA considerations should mention that this
> document registers the MQTP URL scheme.

yup



> References need to be split into normative and informative.

I moved these to informative and left the rest in normative:

[RFC-SHA1] RFC 3184, "US Secure Hash Standard 1 (SHA1)"
[RFC-KEYWORDS] RFC 2119, "Key words for use in RFCs to Indicate 
Requirement Levels"
[RFC-SMTP-TLS] RFC2487, "SMTP Service Extension for Secure SMTP over TLS"



> model-06
> 
> Another 1999 copyright.

oops


> Document needs a security considerations section. It should
> summarize the security considerations raised in the protocol
> documents.

will do.



> That's it!

Excellent!