starred/cypht

Fork 0

mirror of https://github.com/cypht-org/cypht.git synced 2026-04-25 21:15:56 +03:00

[GH-ISSUE #429] UTF8 corruption when replying as text to HTML message part #357

New issue

Closed

opened 2026-02-25 21:34:50 +03:00 by kerem · 8 comments

kerem commented

2026-02-25 21:34:50 +03:00

Owner

Originally created by @jasonmunro on GitHub (Sep 16, 2020).
Original GitHub issue: https://github.com/cypht-org/cypht/issues/429

Originally assigned to: @jasonmunro on GitHub.

Send an HTML email from Gmail with http://kermitproject.org/utf8.html to Cypht
Reply to the HTML part as TEXT and the reply text is mangled.

This only happens when replying to an HTML prt with outbound mail set to TEXT. All other combinations work fine.

Originally created by @jasonmunro on GitHub (Sep 16, 2020). Original GitHub issue: https://github.com/cypht-org/cypht/issues/429 Originally assigned to: @jasonmunro on GitHub. 1. Send an HTML email from Gmail with http://kermitproject.org/utf8.html to Cypht 2. Reply to the HTML part as TEXT and the reply text is mangled. This only happens when replying to an HTML prt with outbound mail set to TEXT. All other combinations work fine.

kerem

2026-02-25 21:34:50 +03:00

closed this issue
added the
bug

core

smtp
labels

kerem commented

2026-02-25 21:34:50 +03:00

Author

Owner

@jasonmunro commented on GitHub (Sep 16, 2020):

Update - actually replying to the HTML part when using outbound of MARKDOWN also fails.

@jasonmunro commented on GitHub (Sep 16, 2020): Update - actually replying to the HTML part when using outbound of MARKDOWN also fails.

kerem commented

2026-02-25 21:34:50 +03:00

Author

Owner

@marclaporte commented on GitHub (Sep 16, 2020):

With 1.9 million downloads, maybe this can help: https://packagist.org/packages/soundasleep/html2text

@marclaporte commented on GitHub (Sep 16, 2020): With 1.9 million downloads, maybe this can help: https://packagist.org/packages/soundasleep/html2text

kerem commented

2026-02-25 21:34:50 +03:00

Author

Owner

@jasonmunro commented on GitHub (Sep 17, 2020):

php: >=7.0

@jasonmunro commented on GitHub (Sep 17, 2020): > php: >=7.0

kerem commented

2026-02-25 21:34:50 +03:00

Author

Owner

@jasonmunro commented on GitHub (Sep 17, 2020):

@marclaporte should be fixed in master. While we can't use the package you posted, it is a good reference for potential issues around HTML -> TEXT conversion. I happened to find this fix elsewhere but still very useful so thanks for the post. I will leave this open for you to confirm on your end the problem is resolved.

Thanks!

@jasonmunro commented on GitHub (Sep 17, 2020): @marclaporte should be fixed in master. While we can't use the package you posted, it is a good reference for potential issues around HTML -> TEXT conversion. I happened to find this fix elsewhere but still very useful so thanks for the post. I will leave this open for you to confirm on your end the problem is resolved. Thanks!

kerem commented

2026-02-25 21:34:50 +03:00

Author

Owner

@marclaporte commented on GitHub (Sep 17, 2020):

About "php: >=7.0": We can use an older version https://packagist.org/packages/soundasleep/html2text#0.5.0 until our requirements increase.

@marclaporte commented on GitHub (Sep 17, 2020): About "php: >=7.0": We can use an older version https://packagist.org/packages/soundasleep/html2text#0.5.0 until our requirements increase.

kerem commented

2026-02-25 21:34:50 +03:00

Author

Owner

@marclaporte commented on GitHub (Sep 17, 2020):

Ah wait, older versions are licensed EPL-1.0, and I don't know off-hand if it's compatible. In any case, we'll sooner or later increase minimum requirements, and the lib will become an option at that point: https://github.com/jasonmunro/cypht/issues/431

@marclaporte commented on GitHub (Sep 17, 2020): Ah wait, older versions are licensed EPL-1.0, and I don't know off-hand if it's compatible. In any case, we'll sooner or later increase minimum requirements, and the lib will become an option at that point: https://github.com/jasonmunro/cypht/issues/431

kerem commented

2026-02-25 21:34:50 +03:00

Author

Owner

@jasonmunro commented on GitHub (Sep 17, 2020):

Well what we have works fine for now and honestly is not very different than what this third party code does.

@jasonmunro commented on GitHub (Sep 17, 2020): Well what we have works fine for now and honestly is not very different than what this third party code does.

kerem commented

2026-02-25 21:34:50 +03:00

Author

Owner

@marclaporte commented on GitHub (Sep 17, 2020):

@jasonmunro understood.

I think we can close this issue. It works for me with the newest code.

Thanks!

@marclaporte commented on GitHub (Sep 17, 2020): @jasonmunro understood. I think we can close this issue. It works for me with the newest code. Thanks!

kerem referenced this issue

2026-02-25 21:36:15 +03:00

[PR #357] [MERGED] Add archive #791