crabmail

commit 48b302f069876beda8f549bc7c27f172f8c2047d
parent f496cab3dd26a3751f38cb9d065ccd78b1a9a879
Author: alex wennerberg <alex@alexwennerberg.com>
Date:   Mon, 28 Mar 2022 19:52:40 -0700

rem panic

Diffstat:
M
TODO
 | 
4
++--
M
src/threading.rs
 | 
5
++++-

2 files changed, 6 insertions(+), 3 deletions(-)
diff --git a/TODO b/TODO
@@ -1,13 +1,13 @@
 TODO
 ====
+audit for security (path bugs, html escape bugs)
 atom get item href working on list
 fix bottom anchor/ latest link
-check for html escape bugz more thoroughly?
 URL encode spaces in links for gemini export
-remove intermediate struct between message -> html. waste of time
 
 later
 ----
+remove intermediate struct between message -> html. waste of time
 Create optional user indexes that link to all their posts
 Duplicate ID verification: warn on duplicate ID, use first received-date. This is to prevent someone overwriting old emails secretly 
 reference mblaze command, add examples to readme
diff --git a/src/threading.rs b/src/threading.rs
@@ -35,7 +35,10 @@ impl ThreadIdx {
     // Todo enumerate errors or something
     // TODO should be format agnostic (use internal representation of email)
     pub fn add_email(&mut self, msg: &Message, path: PathBuf) {
-        let msg_id = msg.get_message_id().unwrap(); // TODO unwrap
+        let msg_id = match msg.get_message_id() {
+            Some(m) => m,
+            None => return,
+        }
         let t = match msg
             .get_received()
             .as_datetime_ref()