From 40b392f1d63f6aaa8794beaff4c0432dd29f1858 Mon Sep 17 00:00:00 2001 From: Vitaly Takmazov Date: Fri, 12 Jan 2018 11:34:43 +0300 Subject: refactoring and tests fixes --- juick-api/src/main/java/com/juick/api/EmailManager.java | 16 ++++++++++------ 1 file changed, 10 insertions(+), 6 deletions(-) (limited to 'juick-api/src/main/java/com/juick/api/EmailManager.java') diff --git a/juick-api/src/main/java/com/juick/api/EmailManager.java b/juick-api/src/main/java/com/juick/api/EmailManager.java index 1c105ef9..c90e7b7a 100644 --- a/juick-api/src/main/java/com/juick/api/EmailManager.java +++ b/juick-api/src/main/java/com/juick/api/EmailManager.java @@ -13,6 +13,7 @@ import org.slf4j.LoggerFactory; import org.springframework.context.ApplicationListener; import org.springframework.stereotype.Component; +import javax.annotation.Nonnull; import javax.inject.Inject; import javax.mail.MessagingException; import javax.mail.Multipart; @@ -29,17 +30,20 @@ import static com.juick.formatters.PlainTextFormatter.formatUrl; @Component public class EmailManager implements ApplicationListener { + + public static final String MSGID_PATTERN = "\\.|@|<"; + private static final Logger logger = LoggerFactory.getLogger(EmailManager.class); @Inject - EmailService emailService; + private EmailService emailService; @Inject - SubscriptionService subscriptionService; + private SubscriptionService subscriptionService; @Inject - MessagesService messagesService; + private MessagesService messagesService; @Inject - UserService userService; + private UserService userService; @Override - public void onApplicationEvent(MessageEvent event) { + public void onApplicationEvent(@Nonnull MessageEvent event) { Message msg = event.getMessage(); if (msg.getRid() == 0) { String subject = String.format("New message from %s", msg.getUser().getName()); @@ -55,7 +59,7 @@ public class EmailManager implements ApplicationListener { } } - void emailNotify(String email, String subject, Message msg) { + private void emailNotify(String email, String subject, Message msg) { Properties prop = System.getProperties(); prop.put("mail.smtp.starttls.enable", "true"); Session session = Session.getDefaultInstance(prop); -- cgit v1.2.3