[Gen-art] Gen-ART last call review of draft-ietf-marf-authfailure-report-07
Alexey Melnikov <alexey.melnikov@isode.com> Sun, 01 January 2012 20:30 UTC
Return-Path: <alexey.melnikov@isode.com>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 2FD611F0C3F; Sun, 1 Jan 2012 12:30:15 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -102.459
X-Spam-Level:
X-Spam-Status: No, score=-102.459 tagged_above=-999 required=5 tests=[AWL=0.140, 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 MfYl5QwSvCMB; Sun, 1 Jan 2012 12:30:14 -0800 (PST)
Received: from rufus.isode.com (cl-125.lon-03.gb.sixxs.net [IPv6:2a00:14f0:e000:7c::2]) by ietfa.amsl.com (Postfix) with ESMTP id 0D4E21F0C36; Sun, 1 Jan 2012 12:30:14 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; t=1325449812; d=isode.com; s=selector; i=@isode.com; bh=QY9TzTH7tsALQjRRaOsXCYO1cRJoM+d74GkAkSpIkuo=; h=From:Sender:Reply-To:Subject:Date:Message-ID:To:Cc:MIME-Version: In-Reply-To:References:Content-Type:Content-Transfer-Encoding: Content-ID:Content-Description; b=izvzH5axKgNB2awf5QszOhvoU7scsj9fU/6QxbfU4DlpN1vUfw68jVAmJIAgsFi9sJT3wt grt+97A2uFSMheyY6cacz1AHGuZ4Mjo2oCH6pWTYq6m/tk3vXFSVLVzcWVeNbTyT4nYpop bBgPpW5kt36m+QrFdp5pTZ8pvpdKVj4=;
Received: from [192.168.0.109] ((unknown) [109.73.6.25]) by rufus.isode.com (submission channel) via TCP with ESMTPSA id <TwDCUQBr10p3@rufus.isode.com>; Sun, 1 Jan 2012 20:30:12 +0000
X-SMTP-Protocol-Errors: NORDNS
Message-ID: <4F00C250.4000508@isode.com>
Date: Sun, 01 Jan 2012 20:30:08 +0000
From: Alexey Melnikov <alexey.melnikov@isode.com>
User-Agent: Mozilla/5.0 (Windows NT 6.1; rv:8.0) Gecko/20111105 Thunderbird/8.0
To: "Hilda L. Fontana" <hilda@hfontana.com>, "Murray S. Kucherawy" <msk@cloudmark.com>, Pete Resnick <presnick@qualcomm.com>
MIME-Version: 1.0
Content-Type: text/plain; charset="windows-1252"; format="flowed"
Content-transfer-encoding: quoted-printable
Cc: gen-art@ietf.org, The IESG <iesg@ietf.org>
Subject: [Gen-art] Gen-ART last call review of draft-ietf-marf-authfailure-report-07
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/gen-art>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Sun, 01 Jan 2012 20:30:15 -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-marf-authfailure-report-07 Reviewer: Alexey Melnikov Review Date: 2012–01–01 IETF LC End Date: 2012-01-04 IESG Telechat date: 2012-01-19 Summary: This draft is almost ready for publication as a standard RFC, but it has some issues that need fixing/discussing. Major issues: I understand that this is a bit pedantic, but ID-nits reports the following: ** Downref: Normative reference to an Experimental RFC: RFC 4408 (ref. 'SPF') and this was not called out during the IETF LC announcement. This reference is truly Normative, so just making it Informative wouldn't work. Minor issues: 1. Introduction [ARF] defines a message format for sending reports of abuse in the messaging infrastructure, with an eye towards automating both the generation and consumption of those reports. There is now also a desire to extend the ARF format to include reporting of messages that fail to authenticate using known authentication methods, as these are sometimes evidence of abuse that can be detected and reported through automated means. The same mechanism can be used to convey forensic information about the specific reason the authentication method failed. Thus, this memo presents such extensions to the Abuse Reporting Format to allow for detailed reporting of message authentication method failures. Maybe that is just me, but when I read "message authentication" I don't really have a clue what you are talking about. I needed to read the rest of the document in order to understand its scope. 2.2. Base 64 base64 is defined in [MIME]. The values that are base64 encodings may contain FWS for formatting purposes as per the usual header field wrapping defined in [MAIL]. During decoding, any characters not in the base64 alphabet are ignored so that such line wrapping does not harm the value. This sentence can be read as allowing other invalid characters outside of FWS. I suggest you reword to make it clear that that is not the case. The ABNF token "FWS" is defined in [DKIM]. 3.1. New ARF Feedback Type: For privacy reasons, report generators might need to redact portions of a reported message such as the end user whose complaint action resulted in the report. See [I-D.IETF-MARF-REDACTION] for a This reference is currently Normative and is a DownRef (as it wasn't called out explicitly during IETF LC). I don't think the reference needs to be Normative: making it Informative will also get rid of the DownRef problem. discussion of this. 3.2.2. Optional For All Reports Delivery-Result: The final message disposition that was enacted by the ADMD generating the report and MUST NOT appear more than once. The first use of acronym ADMD needs expansion and, ideally, an informative reference to where the term is defined. Possible values are: In Section 4: spf-dns = "SPF-DNS:" : { "txt" / "spf" } [FWS] ":" [FWS] domain [FWS] ":" [FWS] quoted-string I think this field is missing CRLF at the end. Also, other fields listed in the same sections are using CFWS, but this one doesn't. Should ABNF for this be aligned with other fields? Nits/editorial comments: None
- [Gen-art] Gen-ART last call review of draft-ietf-… Alexey Melnikov
- Re: [Gen-art] Gen-ART last call review of draft-i… Murray S. Kucherawy
- Re: [Gen-art] Gen-ART last call review of draft-i… Alexey Melnikov