Skip to content

Commit 25fa1a0

Browse files
committed
Attachment name parsing fixed Webklex#410 Webklex#421
1 parent 57e7d28 commit 25fa1a0

File tree

1 file changed

+3
-7
lines changed

1 file changed

+3
-7
lines changed

src/Attachment.php

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -269,12 +269,10 @@ public function decodeName(?string $name): string {
269269
}
270270

271271
$decoder = $this->config['decoder']['message'];
272-
if($decoder === 'utf-8' && extension_loaded('imap')) {
273-
$name = \imap_utf8($name);
274-
}
275-
276272
if (preg_match('/=\?([^?]+)\?(Q|B)\?(.+)\?=/i', $name, $matches)) {
277273
$name = $this->part->getHeader()->decode($name);
274+
} elseif ($decoder === 'utf-8' && extension_loaded('imap')) {
275+
$name = \imap_utf8($name);
278276
}
279277

280278
// check if $name is url encoded
@@ -284,9 +282,7 @@ public function decodeName(?string $name): string {
284282

285283
// sanitize $name
286284
// order of '..' is important
287-
$name = str_replace(['\\', '/', chr(0), ':', '..'], '', $name);
288-
289-
return $name;
285+
return str_replace(['\\', '/', chr(0), ':', '..'], '', $name);
290286
}
291287
return "";
292288
}

0 commit comments

Comments
 (0)