Evaluation: draft-vaudreuil-mdnbis - Message Disposition Notification to Draft Standard

[Bill Fenner <fenner@research.att.com>]

                    Yes    No-Objection  Discuss *  Abstain

Bill Fenner         [   ]     [   ]       [ X ]      [   ]


The "parameter" ABNF in the document body is missing a '"," value'
sequence (which is present in the collected grammar, but not in
the first definition.)
-       parameter = attribute "=" importance *("," value) 
+       parameter = attribute "=" importance "," value *("," value) 


The "-" disease bit the mdn-gateway-field definition:
-          mdn-gateway field = "MDN-Gateway" ":" mta-name-type ";" mta-name
+          mdn-gateway-field = "MDN-Gateway" ":" mta-name-type ";" mta-name
 

AFAIK, single-quotes are not quotes in ABNF.  (This needs to get
fixed in both the document body and the collected ABNF).
      disposition-field =  
                  "Disposition" ":" disposition-mode ";"    
                  disposition-type    
-                 [ '/' disposition-modifier 
+                 [ "/" disposition-modifier 
                  *( "," disposition-modifier ) ] 
 

More "-" disease:
-     should follow the "X-", (e.g. "X Foomail
+     should follow the "X-", (e.g. "X-Foomail-Log-ID" or "X-EDI-info").
 

The mdn-request-header definition in the collected ABNF says that the
mailbox list is seperated by periods; the definition in the document
body says commas.  I selected commas because that is consistent with
the # notation from the original spec.
      mdn-request-header =  
           "Disposition-Notification-To" ":" 
-                 mailbox *("." mailbox) 
+                 mailbox *("," mailbox) 
 

The disposition-type and disposition-modifier in the collected ABNF
disagree with the versions in the body of the document.  I made them
the same as the definitions in the body, but this needs to be double-checked
by someone who knows the intent of the changes. 
      disposition-type = "displayed"    
+                 / "deleted"    
                  / "dispatched"    
                  / "processed"    
-                 / "deleted"    
-                 / "denied"    
-                 / "failed" 
 
-     disposition-modifier =  disposition-modifier-extension 
+     disposition-modifier = "error" / disposition-modifier-extension 
 
 
original-message-id-field got hit by "-" disease:
-     original-message id
+     original-message-id-field = "Original-Message-ID" ":" msg-id


I may have missed some of the places where the editing damaged the original
RFC text.  http://irg.attlabs.net/~fenner/2298bis.html might be useful in
trying to find other problems.