From 79a9f1509c4430f6e65c67e41098ef7aaa227c3f Mon Sep 17 00:00:00 2001 From: Vitaly Takmazov Date: Thu, 4 Jul 2019 14:31:22 +0300 Subject: Render email using templates * do not include tracking pixel in pm * reorganize snapshots layout * reconfigure pebble to not trim anything --- src/main/java/com/juick/server/EmailManager.java | 49 ++++++++++++++++++---- .../server/configuration/WwwAppConfiguration.java | 1 + 2 files changed, 42 insertions(+), 8 deletions(-) (limited to 'src/main/java') diff --git a/src/main/java/com/juick/server/EmailManager.java b/src/main/java/com/juick/server/EmailManager.java index af4ed416..3d17a041 100644 --- a/src/main/java/com/juick/server/EmailManager.java +++ b/src/main/java/com/juick/server/EmailManager.java @@ -7,6 +7,8 @@ import com.juick.service.MessagesService; import com.juick.service.UserService; import com.juick.service.component.*; import com.juick.util.MessageUtils; +import com.mitchellbosecke.pebble.PebbleEngine; +import com.mitchellbosecke.pebble.template.PebbleTemplate; import org.apache.commons.lang3.StringUtils; import org.slf4j.Logger; import org.slf4j.LoggerFactory; @@ -22,9 +24,13 @@ import javax.mail.internet.InternetAddress; import javax.mail.internet.MimeBodyPart; import javax.mail.internet.MimeMessage; import javax.mail.internet.MimeMultipart; +import java.io.IOException; +import java.io.StringWriter; +import java.io.Writer; import java.util.HashMap; import java.util.List; import java.util.Map; +import java.util.Optional; import java.util.Properties; import static com.juick.formatters.PlainTextFormatter.formatPost; @@ -42,6 +48,9 @@ public class EmailManager implements NotificationListener { private MessagesService messagesService; @Inject private UserService userService; + @Inject + private PebbleEngine pebbleEngine; + @Override public void processMessageEvent(@Nonnull MessageEvent event) { Message msg = event.getMessage(); @@ -107,15 +116,9 @@ public class EmailManager implements NotificationListener { headers.put("In-Reply-To", String.format("<%d.%d@juick.com>", original.getMid(), original.getRid())); } } - String plainText = String.format("%s\n\n--\nYou are receiving this because you are subscribed to this user," + - " discussion, tag or mentioned. Reply to this email directly or view it on Juick: %s.", - formatPost(msg), formatUrl(msg)); + String plainText = renderPlaintext(formatPost(msg), formatUrl(msg)).orElseThrow(IllegalStateException::new); String hash = userService.getHashByUID(userService.getUserByEmail(email).getUid()); - String htmlText = String.format("%s

--
You are receiving this because you are subscribed to this user" + - ", discussion, tag or mentioned. Reply to this email directly or view it on Juick." + - "
Configure or disable notifications", - msg.isHtml() ? msg.getText() : MessageUtils.formatHtml(msg), formatUrl(msg), - msg.getMid(), msg.getRid(), hash, hash); + String htmlText = renderHtml(MessageUtils.formatHtml(msg), formatUrl(msg), msg, hash).orElseThrow(IllegalStateException::new); sendEmail(email, subject, plainText, htmlText, headers); } public void sendEmail(String to, String subject, String textPart, String htmlPart, Map messageHeaders) { @@ -162,4 +165,34 @@ public class EmailManager implements NotificationListener { logger.error("mail exception", ex); } } + + public Optional renderPlaintext(String body, String messageUrl) { + PebbleTemplate noteTemplate = pebbleEngine.getTemplate("email/plaintext"); + Map context = new HashMap<>(); + context.put("messageBody", body); + context.put("messageUrl", messageUrl); + try { + Writer writer = new StringWriter(); + noteTemplate.evaluate(writer, context); + return Optional.of(writer.toString()); + } catch (IOException e) { + return Optional.empty(); + } + } + + public Optional renderHtml(String body, String messageUrl, Message msg, String hash) { + PebbleTemplate noteTemplate = pebbleEngine.getTemplate("email/html"); + Map context = new HashMap<>(); + context.put("messageBody", body); + context.put("messageUrl", messageUrl); + context.put("msg", msg); + context.put("hash", hash); + try { + Writer writer = new StringWriter(); + noteTemplate.evaluate(writer, context); + return Optional.of(writer.toString()); + } catch (IOException e) { + return Optional.empty(); + } + } } diff --git a/src/main/java/com/juick/server/configuration/WwwAppConfiguration.java b/src/main/java/com/juick/server/configuration/WwwAppConfiguration.java index 93fe7282..635302c7 100644 --- a/src/main/java/com/juick/server/configuration/WwwAppConfiguration.java +++ b/src/main/java/com/juick/server/configuration/WwwAppConfiguration.java @@ -113,6 +113,7 @@ public class WwwAppConfiguration implements WebMvcConfigurer { .cacheActive(!devToolsArePresent) .extension(springExtension()) .extension(new FormatterExtension()) + .newLineTrimming(false) .strictVariables(true) .build(); } -- cgit v1.2.3