From c2b0d713c355d5c0548fb8a49014159e2c0ec78d Mon Sep 17 00:00:00 2001 From: ralf Date: Wed, 11 May 2022 10:36:43 +0200 Subject: [PATCH] never set a full path as filename of an attachment as it triggers rejection for some mail-servers --- api/src/Mailer.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/api/src/Mailer.php b/api/src/Mailer.php index 1c35365797..d7cb934530 100644 --- a/api/src/Mailer.php +++ b/api/src/Mailer.php @@ -377,7 +377,7 @@ class Mailer extends Horde_Mime_Mail $part->setContents($resource); // setting name, also sets content-disposition attachment (!), therefore we have to do it after "text/calendar; method=" handling - if ($name || !is_resource($data)) $part->setName($name ? $name : Vfs::basename($data)); + if ($name || !is_resource($data)) $part->setName(Vfs::basename($name ?: $data)); // this should not be necessary, because binary data get detected by mime-type, // but at least Cyrus complains about NUL characters