Re: [apps-discuss] Review of draft-marf-spf-reporting-08

Glenn Parsons <glenn.parsons@ericsson.com> Wed, 14 March 2012 03:42 UTC

Return-Path: <glenn.parsons@ericsson.com>
X-Original-To: apps-discuss@ietfa.amsl.com
Delivered-To: apps-discuss@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 5FA3321E800F; Tue, 13 Mar 2012 20:42:55 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -5.96
X-Spam-Level:
X-Spam-Status: No, score=-5.96 tagged_above=-999 required=5 tests=[AWL=0.639, BAYES_00=-2.599, RCVD_IN_DNSWL_MED=-4]
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 Y7K9rBxfVkNS; Tue, 13 Mar 2012 20:42:54 -0700 (PDT)
Received: from imr4.ericy.com (imr4.ericy.com [198.24.6.9]) by ietfa.amsl.com (Postfix) with ESMTP id 8C7F921F84A7; Tue, 13 Mar 2012 20:42:54 -0700 (PDT)
Received: from eusaamw0706.eamcs.ericsson.se ([147.117.20.31]) by imr4.ericy.com (8.14.3/8.14.3/Debian-9.1ubuntu1) with ESMTP id q2E3goqa024867; Tue, 13 Mar 2012 22:42:51 -0500
Received: from EUSAACMS0714.eamcs.ericsson.se ([169.254.2.27]) by eusaamw0706.eamcs.ericsson.se ([147.117.20.31]) with mapi; Tue, 13 Mar 2012 23:42:44 -0400
From: Glenn Parsons <glenn.parsons@ericsson.com>
To: "apps-discuss@ietf.org" <apps-discuss@ietf.org>
Date: Tue, 13 Mar 2012 23:42:43 -0400
Thread-Topic: [apps-discuss] Review of draft-marf-spf-reporting-08
Thread-Index: Ac0Bgoe1oYDzTjrOQCaBqaimNv1vRgAEAMHA
Message-ID: <D9DBDA6E6E3A9F438D9F76F0AF9D7AE34879B0CC18@EUSAACMS0714.eamcs.ericsson.se>
References: <D9DBDA6E6E3A9F438D9F76F0AF9D7AE34879B0CBF1@EUSAACMS0714.eamcs.ericsson.se> <1344986.gzdu38iGHL@scott-latitude-e6320>
In-Reply-To: <1344986.gzdu38iGHL@scott-latitude-e6320>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
acceptlanguage: en-US
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
Cc: "iesg@ietf.org" <iesg@ietf.org>, "draft-ietf-marf-spf-reporting@tools.ietf.org" <draft-ietf-marf-spf-reporting@tools.ietf.org>
Subject: Re: [apps-discuss] Review of draft-marf-spf-reporting-08
X-BeenThere: apps-discuss@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: General discussion of application-layer protocols <apps-discuss.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/apps-discuss>, <mailto:apps-discuss-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/apps-discuss>
List-Post: <mailto:apps-discuss@ietf.org>
List-Help: <mailto:apps-discuss-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/apps-discuss>, <mailto:apps-discuss-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 14 Mar 2012 03:42:55 -0000

If the consensus is that you cannot wait ... my point is that given the IESG note in RFC 4408, it is not appropriate to make this standards track without an indication of the fact that RFC 4408 is being revised/elevated to standards track.  Now that note could be another IESG note...

BTW, would SPFbis would then subsume this RFC ?

On exp / explanation, I misread so that is not an issue.

And on the ABNF, my comment would apply to "ra", "rp" and "rr"

Cheers,
Glenn.


-----Original Message-----
From: Scott Kitterman [mailto:scott@kitterman.com] 
Sent: March-13-12 9:34 PM
To: apps-discuss@ietf.org; Glenn Parsons
Cc: draft-ietf-marf-spf-reporting@tools.ietf.org; iesg@ietf.org
Subject: Re: [apps-discuss] Review of draft-marf-spf-reporting-08

On Tuesday, March 13, 2012 09:01:13 PM Glenn Parsons wrote:
> I have been selected as the Applications Area Directorate reviewer for 
> this draft (for background on appsdir, please see 
> http://trac.tools.ietf.org/area/app/trac/wiki/ApplicationsAreaDirectorate).
> Please resolve these comments along with any other Last Call comments 
> you may receive. Please wait for direction from your document shepherd 
> or AD before posting a new version of the draft. Document:
> draft-ietf-marf-spf-reporting-08

This is not the current revision of the draft.

> Title: SPF Authentication Failure Reporting using the Abuse Report 
> Format
> Reviewer: Glenn Parsons
> Review Date:  March 12, 2012
> 
> Summary:
> This draft is not ready for publication as a Proposed Standard and 
> should be revised before publication Major Issues:
> This is a standards track document that is updating an experimental RFC. 
> And per the IESG Note in RFC 4408 that this is updating, there was 
> quite a controversy on this 6 years ago.  As a result, I do not see 
> how this can be published as a standards track update to the 
> experimental RFC 4408 without some sort of discussion of the issues 
> that led to the initial publication of the RFC 4405-4408 set.  This is 
> especially the case since a 2 year timeline for deployment review was 
> stated as part of the IESG note (and it has been 6 years).  If it is 
> the case that SPF is stable enough to progress on the standards track 
> then I would prefer to see RFC 4408 progressed to standards track 
> before moving forward with these extensions as standards track.  
> Alternatively, if there has been no determination on SPF then it would be more appropriate for this document to have an experimental status.

There is an SPFbis working group that has been chartered to deal with these issues.  The exact question of if it made sense to table this draft until SPFbis was disscussed in the working group and the conclusion was that it did not make sense to wait.

> 5.  The modifier "exp" is not the same as "explanation" in RFC4088.  
> If the intent is for this to be shorter, then a lot more explanation 
> of that (including ABNF update) is required.

Where do you get that it's not the same?  The draft says the source for exp is RFC 4408.

> Minor Issues:

>  3. Does the ABNF really have to be in hex?  That is why not ra 
> instead of
> %x72.61

It's consistent with the related documents the working group has produced.

> 3. The "include: mechanism" is vague.  Suggest adding a reference to 
> clause 5.2 of [SPF] and/or using the same naming convention from RFC
> 4408 -- i.e., "include" mechanism

This seems reasonable.
> 
> Nits:
> 
> 
> ** Downref: Normative reference to an Experimental RFC: RFC 4408 (ref.
> 'SPF')

I think this is addressed.

Scott K