usermin icon indicating copy to clipboard operation
usermin copied to clipboard

Status mail wrong encoding

Open DrCarsonBeckett opened this issue 5 years ago • 12 comments

We're about to release Webmin 1.962 and Usermin 1.811, which are going to include these patches. Within few days, as you install updates, please come back to us sharing how it works for you.

Originally posted by @iliarostovtsev in https://github.com/webmin/webmin/issues/1272#issuecomment-725585499

I need to open another ticket for that too. I received a few mails and the issue is still there. It happens in different browsers (IE, Edge, Firefox, iOS Safari and iOS Firefox) as well on other PCs. It happens in Outlook and Thunderbird as well, the only mail app which displays it correctly for me is the iOS default Mail app.

System: Debian 10 Webmin-Version | 1.962 Usermin-Version | 1.812 Virtualmin Version | 6.14 Authentic theme version | 19.62

Edit: I opened the issue first in the webmin area and closed it (#1332), sorry.

DrCarsonBeckett avatar Nov 22 '20 16:11 DrCarsonBeckett

Hi,

How this issue is different from https://github.com/webmin/usermin/issues/66?

It happens in different browsers (IE, Edge, Firefox, iOS Safari and iOS Firefox) as well on other PCs. It happens in Outlook and Thunderbird as well

What happens?

I opened the issue first in the webmin area and closed it (#1332), sorry.

No problem, thanks!

iliaross avatar Nov 22 '20 16:11 iliaross

I am not sure if its related to that or if its something else. This is about the encoding in the mails and not on the website.

What happens? That the characters in the mail are getting displayed wrong.

This is what gets shown in the mails:

Backup ist komplett. Die Gesamtgröße beträgt

Which should be displayed as:

Backup ist komplett. Die Gesamtgröße beträgt

DrCarsonBeckett avatar Nov 22 '20 17:11 DrCarsonBeckett

I am not sure if its related to that or if its something else. This is about the encoding in the mails and not on the website.

Considering it's generated by Virtualmin, it must be our bug, right @jcameron :slightly_smiling_face:

iliaross avatar Nov 22 '20 23:11 iliaross

Looks like our bug. Can you post the email here as an attachment, with all headers?

jcameron avatar Nov 23 '20 01:11 jcameron

I prefer not to do that due to the fact that data would be public then and I prefer that some of them are not. I can send them however via pm if that's somewhat possible.

DrCarsonBeckett avatar Nov 23 '20 14:11 DrCarsonBeckett

Yes, you can send it to me using my website's form. We will deal with it.

iliaross avatar Nov 23 '20 14:11 iliaross

I just sent it.

DrCarsonBeckett avatar Nov 23 '20 18:11 DrCarsonBeckett

This https://github.com/webmin/webmin/commit/1f9e4795caec8b332678beca2e25b809cddd41b2 should fix this. Could you test it please?

iliaross avatar Nov 30 '20 10:11 iliaross

Tested it, its still the same. From what I read in the patch / note in the commit, its no wonder since the message(-content) itself is displayed wrong and not only the subject. Or did I misunderstood that?

DrCarsonBeckett avatar Nov 30 '20 11:11 DrCarsonBeckett

The odd thing is, that only the backup report mail seems to be wrong. All other emails are encoded correctly (everywhere). Even usermin got issues displaying it correctly, so its not only related to Outlook.

DrCarsonBeckett avatar Dec 02 '20 14:12 DrCarsonBeckett

I looked into this, and it turns out that the code for sending backup status emails in Virtualmin is different and needed to be fixed separately.

jcameron avatar Dec 06 '20 18:12 jcameron

From what I talked with @iliarostovtsev (thanks again for the great support), he said the same. If I understood correctly, you guys added encoding to it. And he said, it should be fixed with the latest virtualmin package (6.14). So far, from the tests we did, it looks good ... at least for me. The last status mail I recieved was also "bug" free on all apps and devices. 😃

DrCarsonBeckett avatar Dec 06 '20 18:12 DrCarsonBeckett