Subject: Re: [dmarc-ietf] XSD concern, policy_evaluated should be required

From: Mike Jones <[email-masked]>
Date: Mon, 2 Dec 2013 11:59:42 -0800

I agree with Greg's point, the min0ccurs should be 1.  What is the purpose of the report without this data included? 

Thanks,
Mike Jones

On Nov 30, 2013, at 10:47 PM, Murray S. Kucherawy <[email-masked]> wrote:

> Do any others wish to comment on this point?
> 
> -MSK
> 
> 
> On Wed, Oct 16, 2013 at 5:31 AM, Greg Colburn <[email-masked]> wrote:
> We noticed some stray aggregate reports without policy_evaluated blocks.  According to page 70 in the current draft, policy_evaluated is minOccurs="0".  Here is the relevant XSD.
> 
>    <xs:complexType name="RowType">
>      <xs:all>
>        <!-- The connecting IP. -->
>        <xs:element name="source_ip" type="IPAddress"/>
>        <!-- The number of matching messages -->
>        <xs:element name="count" type="xs:integer"/>
>        <!-- The DMARC disposition applying to matching
>             messages. -->
>        <xs:element name="policy_evaluated"
>                    type="PolicyEvaluatedType"
>                    minOccurs="0"/>
>      </xs:all>
>    </xs:complexType>
> 
> In my mind the policy_evaluated block is very important, in many ways its the entire point of the report.  It generally answers the question of "what happened".  Without policy_evaluated a significant amount of the value of the data is lost.  I propose that we update the XSD to  set the minOccurs to 1.
> 
> 
>    <xs:complexType name="RowType">
>      <xs:all>
>        <!-- The connecting IP. -->
>        <xs:element name="source_ip" type="IPAddress"/>
>        <!-- The number of matching messages -->
>        <xs:element name="count" type="xs:integer"/>
>        <!-- The DMARC disposition applying to matching
>             messages. -->
>        <xs:element name="policy_evaluated"
>                    type="PolicyEvaluatedType"
>                    minOccurs="1"/>
>      </xs:all>
>    </xs:complexType>
> 
> Sincerely,
> 
> Greg Colburn
> 
> 
> 
> _______________________________________________
> dmarc mailing list
> [email-masked]
> https://www.ietf.org/mailman/listinfo/dmarc
> 
> 
> _______________________________________________
> dmarc mailing list
> [email-masked]
> https://www.ietf.org/mailman/listinfo/dmarc