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

[Raphael OUAZANA <[e-mail]>]

Hi Neil,

Thanks for your comments.

Le 2019-03-21 16:05, Neil Jenkins a écrit :
> 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.

I wrote it in the sense of forbidden by the RFC:
        [...]In any case, in the presence of the $MDNSent keyword,
        the client MUST ignore all other flags or keywords for the
        purpose of generating an MDN and MUST NOT send the MDN.

AlreadyExists seems misleading as the user could just not want to send 
the MDN, and in this case it will never exist. Maybe AlreadyProcessed or 
something like that then?

>> 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?

I wrote a map to "an email sent [...]". Maybe my English is not good 
enough, but for me an email sent means an EmailSubmission. How could I 
clarify?

>> EMAILSUBMISSION/PARSEMDN
> 
> This method is nothing to do with EmailSubmission really. Probably
> should be called MDN/PARSE.

Probably, I hesitated because like this all the MDN stuff is related to 
EmailSubmission, but I can write a MDN/parse if you think it's better.

>> 4.2 [1].  ASKING FOR MDN WHEN SENDING AN EMAIL
> 
> This example is wrong. You can't set a "headers" property [3] in
> Email/set. You need to set it as a specific header, e.g.
> 
> "header:Disposition-Notification-To": "[e-mail]"

Oops sorry, I guess I took it from an old specification, I will fix.

>> 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.

Right, will fix.

>> 5.2 [2].  REGISTRATION OF JMAP KEYWORD '$MDNSENT'
> 
> This keyword is already registered [4]. There is no need to register
> it again (indeed, this is not allowed).

Oh nice, I was not aware of it. I will remove this part.

Regards,
Raphaël.