Hi Alexey, thanks for the review.
> -----Original Message-----
> From: Alexey Melnikov [mailto:[email protected]]
> Sent: Saturday, January 14, 2012 9:21 AM
> To: Hilda L. Fontana; Murray S. Kucherawy; Pete Resnick
> Cc: [email protected]; The IESG
> Subject: Gen-ART last call review of draft-ietf-marf-authfailure-report-09
>
> Document: draft-ietf-marf-authfailure-report-09
> Reviewer: Alexey Melnikov
> Review Date: 2012-01-14
> IETF LC End Date: 2012-01-18
> IESG Telechat date: 2012-01-19
>
> Summary: This draft is ready for publication as a standard RFC, but a
> couple of minor issues remain.
>
> Major issues: None
> Minor issues:
>
> 2.2. Base 64
>
> Sorry for missing this earlier, but RFC 4648, Section 4 is a better
> reference for base64. (Don't forget the section reference, because RFC
> 4648 has 2 base64 alphabets.)
I think that's reasonable.
> In Section 4:
>
> > spf-dns = "SPF-DNS:" : { "txt" / "spf" } [CFWS] ":" [CFWS] domain
> > [CFWS] ":" [CFWS] quoted-string CRLF
>
> I think you are still missing [CFWS] before "txt" and another one
> before CRLF.
>
> Also, you should use "(" and ")" instead of "{" and "}", as the two
> latter are not valid according to ABNF syntax.
>
> To summarize, I think you should use:
>
> spf-dns = "SPF-DNS:" : [CFWS] ( "txt" / "spf" ) [CFWS] ":" [CFWS]
> domain [CFWS] ":" [CFWS] quoted-string [CFWS] CRLF
Almost; the errant unquoted (and thus meaningless) colon before the CFWS has to
come out too. We'll do that for -10.
Thanks again,
-MSK
_______________________________________________
marf mailing list
[email protected]
https://www.ietf.org/mailman/listinfo/marf
