Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[mu4e bug] Regressions with saving images #2516

Open
3 tasks done
jgoerzen opened this issue Jul 11, 2023 · 6 comments
Open
3 tasks done

[mu4e bug] Regressions with saving images #2516

jgoerzen opened this issue Jul 11, 2023 · 6 comments
Labels
bug mu4e specific for mu4e

Comments

@jgoerzen
Copy link

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

  • you are running either an 1.8.x/1.10.x release or master (otherwise please upgrade)
  • you can reproduce the problem without 3rd party extensions (including Doom/Evil, various extensions etc.)
  • you have read all of the above

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

@jgoerzen jgoerzen added bug mu4e specific for mu4e new labels Jul 11, 2023
@djcb djcb removed the new label Jul 14, 2023
@djcb
Copy link
Owner

djcb commented Jul 14, 2023

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
Copy link
Owner

djcb commented Jul 17, 2023

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).

@djcb djcb closed this as completed Jul 17, 2023
@jgoerzen
Copy link
Author

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:

  • M-x image-save: does appear to work, though it doesn't preserve the filename. Doesn't let me save all of them (have to do this for each)
  • M-x mu4e-view-save-attachments: says no attachments for this message
  • M-x mu4e-view-mime-part-action: only lists the text/html and text/plain parts, not the inlined jpegs

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
Copy link
Owner

djcb commented Mar 27, 2024

Can you attach a message where this happens? Thanks!

@jgoerzen
Copy link
Author

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
Copy link
Owner

djcb commented Mar 31, 2024

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!

@djcb djcb reopened this Mar 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug mu4e specific for mu4e
Projects
None yet
Development

No branches or pull requests

2 participants