aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGravatar Vitaly Takmazov2019-07-04 14:31:22 +0300
committerGravatar Vitaly Takmazov2019-07-04 14:31:22 +0300
commit79a9f1509c4430f6e65c67e41098ef7aaa227c3f (patch)
tree8938ab6702c1ce855d4320c715ae09688872b5a8
parent52d88dd00b260c54ba90ab52e283b5b1b1ba851d (diff)
Render email using templates
* do not include tracking pixel in pm * reorganize snapshots layout * reconfigure pebble to not trim anything
-rw-r--r--src/main/java/com/juick/server/EmailManager.java49
-rw-r--r--src/main/java/com/juick/server/configuration/WwwAppConfiguration.java1
-rw-r--r--src/main/resources/templates/email/html.html10
-rw-r--r--src/main/resources/templates/email/plaintext.html5
-rw-r--r--src/test/java/com/juick/server/tests/ServerTests.java36
-rw-r--r--src/test/resources/snapshots/activity/testfollow.json (renamed from src/test/resources/mocks/activity/testfollow.json)0
-rw-r--r--src/test/resources/snapshots/activity/testuser.json (renamed from src/test/resources/mocks/activity/testuser.json)0
-rw-r--r--src/test/resources/snapshots/email/private.html10
-rw-r--r--src/test/resources/snapshots/email/subscription.html10
-rw-r--r--src/test/resources/snapshots/email/subscription.txt5
10 files changed, 116 insertions, 10 deletions
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<br /><br />--<br />You are receiving this because you are subscribed to this user" +
- ", discussion, tag or mentioned. Reply to this email directly or <a href=\"%s\"><img src=\"https://api.juick.com/thread/mark_read/%d-%d.gif?hash=%s\" />view it</a> on Juick." +
- "<br /><a href=\"https://juick.com/settings?hash=%s\">Configure or disable notifications</a>",
- 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<String, String> messageHeaders) {
@@ -162,4 +165,34 @@ public class EmailManager implements NotificationListener {
logger.error("mail exception", ex);
}
}
+
+ public Optional<String> renderPlaintext(String body, String messageUrl) {
+ PebbleTemplate noteTemplate = pebbleEngine.getTemplate("email/plaintext");
+ Map<String, Object> 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<String> renderHtml(String body, String messageUrl, Message msg, String hash) {
+ PebbleTemplate noteTemplate = pebbleEngine.getTemplate("email/html");
+ Map<String, Object> 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();
}
diff --git a/src/main/resources/templates/email/html.html b/src/main/resources/templates/email/html.html
new file mode 100644
index 00000000..086df532
--- /dev/null
+++ b/src/main/resources/templates/email/html.html
@@ -0,0 +1,10 @@
+{{ messageBody | raw }}
+<br />
+<br />
+--
+<br />
+You are receiving this because you are subscribed to this user, discussion, tag or mentioned.
+Reply to this email directly or <a href="{{ messageUrl }}">
+ {% if msg.mid > 0 %}<img src="https://api.juick.com/thread/mark_read/{{ msg.mid }}-{{ msg.rid }}.gif?hash={{ hash }}" />{% endif %}view it</a> on Juick.
+<br />
+<a href="https://juick.com/settings?hash={{ hash }}">Configure or disable notifications</a> \ No newline at end of file
diff --git a/src/main/resources/templates/email/plaintext.html b/src/main/resources/templates/email/plaintext.html
new file mode 100644
index 00000000..a0df0038
--- /dev/null
+++ b/src/main/resources/templates/email/plaintext.html
@@ -0,0 +1,5 @@
+{{ messageBody }}
+
+--
+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: {{ messageUrl }} \ No newline at end of file
diff --git a/src/test/java/com/juick/server/tests/ServerTests.java b/src/test/java/com/juick/server/tests/ServerTests.java
index 4c72913e..de742447 100644
--- a/src/test/java/com/juick/server/tests/ServerTests.java
+++ b/src/test/java/com/juick/server/tests/ServerTests.java
@@ -28,6 +28,7 @@ import com.gargoylesoftware.htmlunit.html.DomElement;
import com.gargoylesoftware.htmlunit.html.HtmlPage;
import com.jayway.jsonpath.JsonPath;
import com.juick.*;
+import com.juick.formatters.PlainTextFormatter;
import com.juick.model.AnonymousUser;
import com.juick.model.CommandResult;
import com.juick.model.PrivateChats;
@@ -218,10 +219,16 @@ public class ServerTests {
@Inject
private Profile profileController;
- @Value("classpath:mocks/activity/testuser.json")
+ @Value("classpath:snapshots/activity/testuser.json")
private Resource testuserResponse;
- @Value("classpath:mocks/activity/testfollow.json")
+ @Value("classpath:snapshots/activity/testfollow.json")
private Resource testfollowRequest;
+ @Value("classpath:snapshots/email/subscription.html")
+ private Resource testSubscriptionHtmlEmail;
+ @Value("classpath:snapshots/email/private.html")
+ private Resource testPrivateHtmlEmail;
+ @Value("classpath:snapshots/email/subscription.txt")
+ private Resource testSubscriptionTextEmail;
@Value("classpath:static/av-96.png")
private Resource defaultAvatar;
@Value("classpath:cmyk.jpg")
@@ -241,6 +248,8 @@ public class ServerTests {
@Inject
private KeystoreManager testKeystoreManager;
+ @Inject
+ private EmailManager emailManager;
@Inject
private ApplicationEventPublisher applicationEventPublisher;
@@ -2121,4 +2130,27 @@ public class ServerTests {
.findFirst().orElseThrow(IllegalStateException::new);
assertThat(rareTagStats.getUsageCount(), is(1));
}
+ private String getSnapshot(Resource resource) throws IOException {
+ return IOUtils.toString(resource.getInputStream(), StandardCharsets.UTF_8);
+ }
+ @Test
+ public void emailTemplatesTest() throws IOException {
+ String plainText = emailManager.renderPlaintext("yo", "https://localhost/m/1").orElseThrow();
+ assertThat(plainText, is(getSnapshot(testSubscriptionTextEmail)));
+ User demo = MockUtils.mockUser(45, ugnichName, ugnichPassword);
+ Message html = MockUtils.mockMessage(56, demo, "yo");
+ String htmlText = emailManager.renderHtml(
+ MessageUtils.formatHtml(html),
+ PlainTextFormatter.formatUrl(html),
+ html, "12345")
+ .orElseThrow();
+ assertThat(htmlText, is(getSnapshot(testSubscriptionHtmlEmail)));
+ html.setMid(0);
+ String htmlPM = emailManager.renderHtml(
+ MessageUtils.formatHtml(html),
+ PlainTextFormatter.formatUrl(html),
+ html, "12345")
+ .orElseThrow();
+ assertThat(htmlPM, is(getSnapshot(testPrivateHtmlEmail)));
+ }
}
diff --git a/src/test/resources/mocks/activity/testfollow.json b/src/test/resources/snapshots/activity/testfollow.json
index e308e52e..e308e52e 100644
--- a/src/test/resources/mocks/activity/testfollow.json
+++ b/src/test/resources/snapshots/activity/testfollow.json
diff --git a/src/test/resources/mocks/activity/testuser.json b/src/test/resources/snapshots/activity/testuser.json
index 95fc2aa9..95fc2aa9 100644
--- a/src/test/resources/mocks/activity/testuser.json
+++ b/src/test/resources/snapshots/activity/testuser.json
diff --git a/src/test/resources/snapshots/email/private.html b/src/test/resources/snapshots/email/private.html
new file mode 100644
index 00000000..fee506c7
--- /dev/null
+++ b/src/test/resources/snapshots/email/private.html
@@ -0,0 +1,10 @@
+<b>@ugnich:</b></br/><i></i><br/>yo<br />
+<br />
+<br />
+--
+<br />
+You are receiving this because you are subscribed to this user, discussion, tag or mentioned.
+Reply to this email directly or <a href="https://juick.com/pm/inbox">
+ view it</a> on Juick.
+<br />
+<a href="https://juick.com/settings?hash=12345">Configure or disable notifications</a> \ No newline at end of file
diff --git a/src/test/resources/snapshots/email/subscription.html b/src/test/resources/snapshots/email/subscription.html
new file mode 100644
index 00000000..c75360ed
--- /dev/null
+++ b/src/test/resources/snapshots/email/subscription.html
@@ -0,0 +1,10 @@
+<b>@ugnich:</b></br/><i></i><br/>yo<br />
+<br />
+<br />
+--
+<br />
+You are receiving this because you are subscribed to this user, discussion, tag or mentioned.
+Reply to this email directly or <a href="https://juick.com/m/56">
+ <img src="https://api.juick.com/thread/mark_read/56-0.gif?hash=12345" />view it</a> on Juick.
+<br />
+<a href="https://juick.com/settings?hash=12345">Configure or disable notifications</a> \ No newline at end of file
diff --git a/src/test/resources/snapshots/email/subscription.txt b/src/test/resources/snapshots/email/subscription.txt
new file mode 100644
index 00000000..57967c4b
--- /dev/null
+++ b/src/test/resources/snapshots/email/subscription.txt
@@ -0,0 +1,5 @@
+yo
+
+--
+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: https://localhost/m/1 \ No newline at end of file