fix(backend): attachments with PDF files (and probably other subtypes…

… under the application type) (#1442)
cypht-org · Feb 6, 2025 · 18965ef · 18965ef
1 parent 13e3a96
commit 18965ef
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/modules/smtp/modules.php b/modules/smtp/modules.php
@@ -190,9 +190,9 @@ public function process() {
                     foreach ($msg_struct[0]['subs'] as $ind => $sub) {
                         if ($ind != '0.1') {
                             $new_attachment['basename'] = $sub['description'];
-                            $new_attachment['name'] = $sub['attributes']['name'];
+                            $new_attachment['name'] = $sub['attributes']['name'] ?? $sub['attributes']['filename']; // some file types (i.e. pdf) use the filename attribute instead of name.
                             $new_attachment['size'] = $sub['size'];
-                            $new_attachment['type'] = $sub['type'];
+                            $new_attachment['type'] = $sub['type'] . "/" . $sub['subtype'];
                             $file_path = $file_dir . $new_attachment['name'];
                             $content = Hm_Crypt::ciphertext($mailbox->get_message_content(hex2bin($path[2]), $this->request->get['uid'], $ind), Hm_Request_Key::generate());
                             file_put_contents($file_path, $content);