Re: draft-ietf-msgtrk-mtqp-04.txt and draft-ietf-msgtrk-model-04.txt

[Tony Hansen <tony@xxxxxxx>]

always some nits. :-) thanks Greg!

more below.

	Tony

Gregory Neil Shapiro wrote:
> 
> tony> http://www.geocities.com/tonylhansen/draft-ietf-msgtrk-mtqp-04.txt
> 
> Here are my comments:
> 
> 1. Replace 'A record' with 'address record':
> 
> -record pointing at the server host, and if not, will have an A record.
> +record pointing at the server host, and if not, will have an address record.

thought I had gotten all of those. rats.

> 2. Spelling errors:
> ...
> 
> 3. The firewall examples showing relaying to another host and then that
>    host's responses seem to show the wrong Remote-MTA hostname but I could
>    be wrong.  Somebody should double chek my work:
> 
> @@ -519,7 +519,7 @@
>       S: Final-Recipient: rfc822; user1@xxxxxxxxxxxx
>       S: Action: relayed
>       S: Status: 2.1.9
> -     S: Remote-MTA: dns; example2.com
> +     S: Remote-MTA: dns; smtp.example3.com
>       S: Last-Attempt-Date: Mon,  1 Jan 2001 19:15:03 -0500
>       S:
>       S: --%%%%
> @@ -553,7 +553,7 @@
>       S: Final-Recipient: rfc822; user1@xxxxxxxxxxxx
>       S: Action: relayed
>       S: Status: 2.1.9
> -     S: Remote-MTA: dns; example2.com
> +     S: Remote-MTA: dns; smtp.example3.com
>       S: Last-Attempt-Date: Mon,  1 Jan 2001 19:15:03 -0500
>       S:

I think you're right.
 
> 4. The MTQP option registration for STARTTLS is improperly formatted:
> 
>       One MTQP option is defined in this document:
> -     option identifier: STARTTLS option parameters: none added commands:
> -     STARTTLS standard commands affected: none specification reference:
> -     RFC TBD discussion: see RFC TBD
> +     option identifier: STARTTLS
> +     option parameters: none
> +     added commands: STARTTLS
> +     standard commands affected: none
> +     specification reference: RFC TBD
> +     discussion: see RFC TBD

left off the ".nf/.fi" pair, for those of you who speak *roff.