Re: [Jmap] I-D Action: draft-ietf-jmap-mdn-01.txt

["Neil Jenkins" <[e-mail]>]

A few comments on the new draft:

>   The client SHOULD NOT issue a sendMDN request if the message has the
   "$MDNSent" keyword set.  In this case, the server MUST reject the
   submission with a standard "forbiddenToSend" SetError.

I don't think this should be a `forbiddenToSend` error; that implies the user does not have permission to send. An `alreadyExists` error might be appropriate, or some kind of new "precondition-failed" type error.

>  o  *created*: "String[EmailSubmission]" A map of creation id (client-
      specified) to an email sent from the referenced properties.

The type says this is a map to EmailSubmission objects, but the description says it is a map to an Email. Which is it?

> *
> EmailSubmission/parseMDN

*

This method is nothing to do with EmailSubmission really. Probably should be called *MDN/parse*.

> *
> 4.2 <https://tools.ietf.org/html/draft-ietf-jmap-mdn-01#section-4.2>.  Asking for MDN when sending an email

*

This example is wrong. You can't set a "headers" property <https://jmap.io/spec-mail.html#email/set> in Email/set. You need to set it as a specific header, e.g.

"header:Disposition-Notification-To": "[e-mail]"

>  The client issues a parse request:

            [[ "EmailSubmission/parseMDN", {
              "accountId": "ue150411c",
              "blobIds: "0f9f65ab-dc7b-4146-850f-6e4881093965"
            }, "0" ]]

This blobIds argument should be an array, but the example is just passing a string.

> *
> 5.2 <https://tools.ietf.org/html/draft-ietf-jmap-mdn-01#section-5.2>.  Registration of JMAP keyword '$MDNSent'

*

This keyword is already registered <https://www.iana.org/assignments/imap-jmap-keywords/imap-jmap-keywords.xhtml>. There is no need to register it again (indeed, this is not allowed).

Cheers,
Neil.