Gen-ART review of draft-ietf-appsawg-malformed-mail-09
"Black, David" <david.black@emc.com> Sat, 02 November 2013 01:32 UTC
Return-Path: <david.black@emc.com>
X-Original-To: ietf@ietfa.amsl.com
Delivered-To: ietf@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id B1C6211E80E9; Fri, 1 Nov 2013 18:32:47 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -102.599
X-Spam-Level:
X-Spam-Status: No, score=-102.599 tagged_above=-999 required=5 tests=[BAYES_00=-2.599, USER_IN_WHITELIST=-100]
Received: from mail.ietf.org ([12.22.58.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id azARdMI8AdOg; Fri, 1 Nov 2013 18:32:43 -0700 (PDT)
Received: from mailuogwdur.emc.com (mailuogwdur.emc.com [128.221.224.79]) by ietfa.amsl.com (Postfix) with ESMTP id 6053F11E80E7; Fri, 1 Nov 2013 18:32:43 -0700 (PDT)
Received: from maildlpprd55.lss.emc.com (maildlpprd55.lss.emc.com [10.106.48.159]) by mailuogwprd51.lss.emc.com (Sentrion-MTA-4.3.0/Sentrion-MTA-4.3.0) with ESMTP id rA21WXvh031172 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Fri, 1 Nov 2013 21:32:33 -0400
X-DKIM: OpenDKIM Filter v2.4.3 mailuogwprd51.lss.emc.com rA21WXvh031172
DKIM-Signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d=emc.com; s=jan2013; t=1383355954; bh=vFy9Uc7OwUUY6NR5/3KmfYthlb0=; h=From:To:CC:Date:Subject:Message-ID:Content-Type: Content-Transfer-Encoding:MIME-Version; b=YYSo0+oB9/4ThNeBBEccASLa7pGuEvsT/5Efhe32fPanFha1AmGPBiX2IIJQxDoQ2 hAf5CgVfjYALlQ0OUAqZQRJ1sKYMTgQKDQlwPYKHek/LSc4A9wQ4fKJ7q13ZBcoZQ1 qeQSbyoEMwlApbGzJhgc5koc4ixLRSq75yqKGFbU=
X-DKIM: OpenDKIM Filter v2.4.3 mailuogwprd51.lss.emc.com rA21WXvh031172
Received: from mailusrhubprd03.lss.emc.com (mailusrhubprd03.lss.emc.com [10.253.24.21]) by maildlpprd55.lss.emc.com (RSA Interceptor); Fri, 1 Nov 2013 21:32:25 -0400
Received: from mxhub10.corp.emc.com (mxhub10.corp.emc.com [10.254.92.105]) by mailusrhubprd03.lss.emc.com (Sentrion-MTA-4.3.0/Sentrion-MTA-4.3.0) with ESMTP id rA21WOUm014873 (version=TLSv1/SSLv3 cipher=AES128-SHA bits=128 verify=FAIL); Fri, 1 Nov 2013 21:32:24 -0400
Received: from mx15a.corp.emc.com ([169.254.1.239]) by mxhub10.corp.emc.com ([10.254.92.105]) with mapi; Fri, 1 Nov 2013 21:32:24 -0400
From: "Black, David" <david.black@emc.com>
To: "Murray S. Kucherawy (superuser@gmail.com)" <superuser@gmail.com>, "gshapiro@proofpoint.com" <gshapiro@proofpoint.com>, "ned.freed@mrochek.com" <ned.freed@mrochek.com>, "General Area Review Team (gen-art@ietf.org)" <gen-art@ietf.org>
Date: Fri, 01 Nov 2013 21:32:23 -0400
Subject: Gen-ART review of draft-ietf-appsawg-malformed-mail-09
Thread-Topic: Gen-ART review of draft-ietf-appsawg-malformed-mail-09
Thread-Index: Ac7Xa12dzFVGhnZdR0m8P/d5wkHADg==
Message-ID: <8D3D17ACE214DC429325B2B98F3AE712026AAEBA81@MX15A.corp.emc.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
acceptlanguage: en-US
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-Sentrion-Hostname: mailusrhubprd03.lss.emc.com
X-RSA-Classifications: public
Cc: "Black, David" <david.black@emc.com>, "Barry Leiba (barryleiba@computer.org)" <barryleiba@computer.org>, "ietf@ietf.org" <ietf@ietf.org>, "apps-discuss@ietf.org" <apps-discuss@ietf.org>
X-BeenThere: ietf@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: IETF-Discussion <ietf.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ietf>, <mailto:ietf-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/ietf>
List-Post: <mailto:ietf@ietf.org>
List-Help: <mailto:ietf-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ietf>, <mailto:ietf-request@ietf.org?subject=subscribe>
X-List-Received-Date: Sat, 02 Nov 2013 01:32:47 -0000
I am the assigned Gen-ART reviewer for this draft. For background on Gen-ART, please see the FAQ at <http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>. Please resolve these comments along with any other Last Call comments you may receive. Document: draft-ietf-appsawg-malformed-mail-09 Reviewer: David L. Black Review Date: November 1, 2013 IETF LC End Date: October 29, 2013 IESG Telechat date: November 21, 2013 Summary: This draft is basically ready for publication, but has nits that should be fixed before publication. This draft discusses common errors in mail message syntax and provides useful guidance on handling them. The draft is well written and likely to be of significant value to implementers. Most of my comments relate to clarity, but none of them seem important enough to be characterized as issues that really have to be fixed, although the sloppy terminology usage in Sections 7.1.* comes close. I apologize for this review appearing slightly after the end of IETF Last Call - there is too much going on in the weeks immediately before this IETF meeting. Nits/editorial comments: The word "naked" is used a few times to refer to something that occurs in isolation, without enclosure in another construct, e.g., a naked CR. This idiomatic use of "naked" should be explained before it is used. In Section 1.1, I have always heard Postel's law as: - Be conservative in what you send, and - Be liberal in what you accept. The change from "do" in this draft to "send" (above) seems useful, as it should help focus the discussion in the second paragraph of Section 1.1 - Postel's law, as I have understood it, has never blessed anything remotely resembling there being "no limits to the liberties that a sender might take." Section 5's section title "Mail Submission Agents" doesn't seem to be connected to its MHS and MTA content. It would be useful to add a sentence to remind readers, including this one ;-), of what Mail Submission Agents are. Sections 7.1.* offers degrees of advice qualified by "safely", "usually", "reasonably" and "should". There appear to be only two concepts: - "safely": Do this all the time. - "usually", "reasonably", "should": This is the recommended course of action, but there may be exceptions. While RFC 2119 is not intended for Informational documents, this is an example of the sort of sloppiness that RFC 2119 is intended to clean up. At the very least, the use of three words for essentially the same concept is poor form, and RFC 2119 can be used in an Informational document when appropriate caveats are provided in the terminology section that references it. In Section 7.1.4, "Likewise" is not a good way to associate the second example with the first, because it handles the missing parenthesis in a rather different fashion (adds quotes instead of inserting the missing parenthesis character). In Section 7.7, the first use of "8bit" occurs in "8bit material" but some of the subsequent occurrences omit the word "material" - that word should be used with all occurrences. idnits 2.13.00 generated a couple of warnings about obsolete references: -- Obsolete informational reference (is this intentional?): RFC 1113 (ref. 'PEM') (Obsoleted by RFC 1421) -- Obsolete informational reference (is this intentional?): RFC 733 (Obsoleted by RFC 822) In both cases, the reference appears to be intentional, and the warning should be ignored. Thanks, --David ---------------------------------------------------- David L. Black, Distinguished Engineer EMC Corporation, 176 South St., Hopkinton, MA 01748 +1 (508) 293-7953 FAX: +1 (508) 293-7786 david.black@emc.com Mobile: +1 (978) 394-7754 ----------------------------------------------------
- Gen-ART review of draft-ietf-appsawg-malformed-ma… Black, David
- Re: Gen-ART review of draft-ietf-appsawg-malforme… Murray S. Kucherawy
- Re: Gen-ART review of draft-ietf-appsawg-malforme… Barry Leiba
- RE: Gen-ART review of draft-ietf-appsawg-malforme… Black, David
- Re: Gen-ART review of draft-ietf-appsawg-malforme… John Levine