never set a full path as filename of an attachment as it triggers rejection for some mail-servers

This commit is contained in:
ralf 2022-05-11 10:36:43 +02:00
parent c5cab2e7b3
commit c2b0d713c3

View File

@ -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