diff options
author | Manos Pitsidianakis <el13635@mail.ntua.gr> | 2019-09-27 22:21:35 +0300 |
---|---|---|
committer | Manos Pitsidianakis <el13635@mail.ntua.gr> | 2019-09-27 22:21:35 +0300 |
commit | 250129665b0a74d0946cbfd39d3dcc36e62a67d4 (patch) | |
tree | af4ac292b3f801957bfa43ba6b1c07fd0b5974e1 /ui/src/components/mail/view.rs | |
parent | 19ec6e54fcffdb896a95222bbd7132f2e505a073 (diff) |
Pass attachment names through decoding
Attachment names in Content-Type parameters can be encoded (eg
=?UTF-8...), so try decoding with phrase() first
Diffstat (limited to 'ui/src/components/mail/view.rs')
-rw-r--r-- | ui/src/components/mail/view.rs | 17 |
1 files changed, 11 insertions, 6 deletions
diff --git a/ui/src/components/mail/view.rs b/ui/src/components/mail/view.rs index 529d8799..8a8e7c5f 100644 --- a/ui/src/components/mail/view.rs +++ b/ui/src/components/mail/view.rs @@ -875,13 +875,18 @@ impl Component for MailView { ContentType::Other { ref name, .. } => { let attachment_type = u.mime_type(); let binary = query_default_app(&attachment_type); + let mut name_opt = name.as_ref().and_then(|n| { + melib::email::parser::phrase(n.as_bytes()) + .to_full_result() + .ok() + .and_then(|n| String::from_utf8(n).ok()) + }); + if name_opt.is_none() { + name_opt = name.as_ref().map(|n| n.clone()); + } if let Ok(binary) = binary { - let p = create_temp_file( - &decode(u, None), - name.as_ref().map(|n| n.clone()), - None, - true, - ); + let p = + create_temp_file(&decode(u, None), name_opt, None, true); Command::new(&binary) .arg(p.path()) .stdin(Stdio::piped()) |