[JAMES-3566] EmailSubmission/set does an Email/set implicit call when no onSuccess properties

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.6.0
• Fix Version/s: 3.7.0, 3.6.1
• Component/s: JMAP
• Assignee: Antoine Duprat
• Reporter: Benoit Tellier
• Created: 16/Apr/21 16:15
• Updated: 27/Apr/21 03:03
• Resolved: 27/Apr/21 03:03

Description

{
  "using": ["urn:ietf:params:jmap:core", "urn:ietf:params:jmap:mail", "urn:ietf:params:jmap:submission"],
  "methodCalls": [
     ["EmailSubmission/set", {
       "accountId": "$ACCOUNT_ID",
       "create": {
         "k1490": {
           "emailId": "${messageId.serialize}",
           "envelope": {
             "mailFrom": {"email": "${BOB.asString}"},
             "rcptTo": [{"email": "${ANDRE.asString}"}]
           }
         }
    }
  }, "c1"]]
}

Returns:

{
    "sessionState": "2c9f1b12-b35a-43e6-9af2-0106fb53a943",
    "methodResponses": [
        [
            "EmailSubmission/set",
            {
                "accountId": "29883977c13473ae7cb7678ef767cbfbaffc8a44a6e463d971d23a65c1dc4af6",
                "newState": "2c9f1b12-b35a-43e6-9af2-0106fb53a943",
                "notCreated": {
                    "k1490": {
                        "type": "forbiddenMailFrom",
                        "description": "Attempt to send a mail whose MimeMessage From and Sender fields not allowed for connected user: List([e-mail])"
                    }
                }
            },
            "c1"
        ],
        [
            "Email/set",
            {
                "accountId": "29883977c13473ae7cb7678ef767cbfbaffc8a44a6e463d971d23a65c1dc4af6",
                "oldState": "668a394e-a324-4d3d-bef0-945ae79f18c8",
                "newState": "668a394e-a324-4d3d-bef0-945ae79f18c8"
            },
            "c1"
        ]
    ]
}

The empty Email/set call should not be performed when no onSuccessXXX properties is specified.

Thus this would be more appropriate:

{
    "sessionState": "2c9f1b12-b35a-43e6-9af2-0106fb53a943",
    "methodResponses": [
        [
            "EmailSubmission/set",
            {
                "accountId": "29883977c13473ae7cb7678ef767cbfbaffc8a44a6e463d971d23a65c1dc4af6",
                "newState": "2c9f1b12-b35a-43e6-9af2-0106fb53a943",
                "notCreated": {
                    "k1490": {
                        "type": "forbiddenMailFrom",
                        "description": "Attempt to send a mail whose MimeMessage From and Sender fields not allowed for connected user: List([e-mail])"
                    }
                }
            },
            "c1"
        ]
    ]
}

Reported by Tung on Gitter...

Issue Links

• links to: GitHub Pull Request #394