Merge branch 'feature/mailbox' of dlab/hive

from pull-request 1501 * refs/heads/feature/mailbox: Mailbox: Ignore emails from a person who is not a Yobi user Reviewed-by: 채수원 <sw.chae@navercorp.com>
naver · Mar 2, 2015 · a6d17fc · a6d17fc
2 parents cb43871 + adabebb
commit a6d17fc
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/app/mailbox/EmailHandler.java b/app/mailbox/EmailHandler.java
@@ -217,6 +217,10 @@ private static void handleMessage(@Nonnull IMAPMessage msg) {
 
             author = User.findByEmail(senderAddress.getAddress());
 
+            if (author.isAnonymous()) {
+                continue;
+            }
+
             try {
                 createResources(msg, author, errors);
             } catch (MailHandlerException e) {

diff --git a/docs/technical/mailbox.md b/docs/technical/mailbox.md
@@ -39,7 +39,7 @@ Mailbox handles the fetched emails immediately. See "Handling the emails".
 Yobi posts the fetched emails as an issue or a comment if possible.
 
 Mailbox determines the author by the sender's email address from 'From' header of
-the email.
+the email. Emails from the sender who is not a user of Yobi are ignored.
 
 Mailbox determines the projects by the detail parts, which comes after plus
 sign in local part, e.g. owner/project from yobi+owner/project@mail.com, of the