From 8f15b804d1a760d4f4e5c0c47a8a26ee2c04a830 Mon Sep 17 00:00:00 2001
From: Vitaly Ovchinnikov <v@postbox.nz>
Date: Tue, 12 Sep 2023 21:19:12 +0000
Subject: hyperlinks: better parsing of emails without mailto prefixes

Add some new tests from the emails I have and make them work by
adjusting the code that looks for hyperlinks.

The idea is to treat "inline" emails (those without mailto:) a little
bit different and stop a little earlier while looking for their ends.

Signed-off-by: Vitaly Ovchinnikov <v@postbox.nz>
Acked-by: Robin Jarry <robin@jarry.cc>
---
 lib/parse/hyperlinks_test.go | 20 ++++++++++++++++++++
 1 file changed, 20 insertions(+)

(limited to 'lib/parse/hyperlinks_test.go')

diff --git a/lib/parse/hyperlinks_test.go b/lib/parse/hyperlinks_test.go
index cedad648..00a06764 100644
--- a/lib/parse/hyperlinks_test.go
+++ b/lib/parse/hyperlinks_test.go
@@ -114,6 +114,26 @@ func TestHyperlinks(t *testing.T) {
 			text:  "You can reach me via the somewhat strange, but nonetheless valid, email mailto:~mpldr/list@[2001:db8::7]?subject=whazzup%3F",
 			links: []string{"mailto:~mpldr/list@[2001:db8::7]?subject=whazzup%3F"},
 		},
+		{
+			name:  "simple email in <a href>",
+			text:  `<a href="mailto:a@abc.com" rel="noopener noreferrer">`,
+			links: []string{"mailto:a@abc.com"},
+		},
+		{
+			name:  "simple email in <a> body",
+			text:  `<a href="#" rel="noopener noreferrer">a@abc.com</a><br/><p>more text</p>`,
+			links: []string{"mailto:a@abc.com"},
+		},
+		{
+			name:  "emails in <a> href and body",
+			text:  `<a href="mailto:a@abc.com" rel="noopener noreferrer">b@abc.com</a><br/><p>more text</p>`,
+			links: []string{"mailto:a@abc.com", "mailto:b@abc.com"},
+		},
+		{
+			name:  "email in &lt;...&gt;",
+			text:  `<div>01.02.2023, 10:11, "Firstname Lastname" &lt;a@abc.com&gt;:</div>`,
+			links: []string{"mailto:a@abc.com"},
+		},
 	}
 
 	for i, test := range tests {
-- 
cgit