djcb / mu

maildir indexer/searcher + emacs mail client + guile bindings
http://www.djcbsoftware.nl/code/mu
GNU General Public License v3.0
1.61k stars 391 forks source link

[mu4e bug] Regressions with saving images #2516

Open jgoerzen opened 1 year ago

jgoerzen commented 1 year ago

Describe the bug

After upgrading from mu4e 1.4.15 to mu4e 1.8.14, I am encountering these issues:

  1. There is no way to save all images like we used to have in the mu4e view.
  2. While I can use the action menu to save attachments, I must use M-x image-save to save inline images.
  3. M-x image-save does not preserve the filename. In fact, I can no longer even access the filenames of inline images.

There are some related issues:

How to Reproduce

Receive a message with inline image attachments.

Observe they are not available for save in the action menu, and filenames are not preserved with M-x image-save. Also there is no way to save all attachments, regardless of type, anymore.

Environment Debian bookworm with Doom emacs and mu4e 1.8.14.

Checklist

Please make sure you all items in the checklist are set/met before filing the ticket. Thank you!

djcb commented 1 year ago

Yeah, with the move to the gnus-based viewer we gained many features, and lost a few, which are slowly coming back... I'm planning to update the mime-handling code for the next stable release (1.11.x --> 1.12)

djcb commented 1 year ago

A first version of this has landed in master, and it does bring back saving for disposition "inline" attachments and some other improvements; so closing this.

But note, unfortunately this is too big to be back-merged into the release branches for 1.10 (let alone 1.8).

jgoerzen commented 6 months ago

Hi, Thanks for your work on this!

I've updgraded to 1.12.2 and still having difficulties. I looked at https://www.djcbsoftware.nl/code/mu/mu4e/MSGV-Attachments-and-MIME_002dparts.html and tried:

So I must be missing something?

Also it initially displays the images way too big, but if I click on the text/html at the top -- even though it is already displaying that -- it will zoom them out to fit in the window.

djcb commented 6 months ago

Can you attach a message where this happens? Thanks!

jgoerzen commented 6 months ago

It has some private stuff in it, so I've sent you an example by email to the address listed in the AUTHORS file.

djcb commented 6 months ago

Ah, yeah, that's a tricky message. I'll reopen this.

Note that all of the above work for most messages, it's just some very specific message. That's why I always ask to attach an example message to reproduce problems (of course removing any confidential info); anyway, I have such an example now, but please remember for any future cases, thanks!