Re: [spfbis] WGLC: draft-ietf-spfbis-4408bis-14 - Fully parse record *first*

[Stuart Gathman <[e-mail]>]

On 04/17/2013 12:50 AM, S Moonesamy wrote:
>
> As Scott mentioned, things have been very quiet for this WGLC.  It 
> helps if there are people who read the draft as you did above as I can 
> determine whether the working group reviewed the draft and is ok with it.
Minor nit:

Section 5.1

Mechanisms listed after "all" MUST be ignored.

Sure, section 4.6 says

If there are any syntax errors
    anywhere in the record, check_host() returns immediately with the
    result "permerror", without further interpretation.

But an implementer could misinterpret this as saying the following 
should get Fail rather than PermError:

v=spf1 mx -all foobar

Section 4.6 doesn't make it clear you have to parse everything 
(returning permerror on syntax errors), and only *then* interpret. The 
wording makes it sound like you could parse and interpret one term at a 
time, stopping when you get a match or syntax error.