[GitHub] [james-project] chibenwa commented on pull request #385: JAMES-3520 MDN/send

[GitBox <gi...@apache.org>]

chibenwa commented on pull request #385:
URL: https://github.com/apache/james-project/pull/385#issuecomment-822190733


   ```
   1. RFC write the IdentityId used to define a final recipient. But I don't find any reference about IdentityId on the source code. So, How I can define it?
   ```
   
   -> IdentityGetMethod
   
   We *may* need to extract a subclass `IdentifyStore` that resolves Identities, so that other methods can access it.
   
   ```
   2. How I can control about rate limit (or over quota)?
   ```
   
   Rate limiting is out of the scope of our current efforts to support JMAP. Do not try.
   
   Rate limiting in distributed systems is a complex topic. James is likely a bad place to enforce it.
   
   ```
   3. I have an MDNSendMethod.recordCreationIdInProcessingContext, but I don't sure it is necessary. Should I remove this?
   ```
   
   In the case of a `send` we do not keep a local copy of the MDN so from a JMAP perspective, we did not create anything. You can remove it (for MDN/send only)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@james.apache.org
For additional commands, e-mail: notifications-help@james.apache.org