Skip to content

Allow to set HTML Content-Transfer-Encoding based on UseMIMEfor 8bit - #8477

Closed
bojovyletoun wants to merge 1 commit into
roundcube:masterfrom
bojovyletoun:patch-1
Closed

Allow to set HTML Content-Transfer-Encoding based on UseMIMEfor 8bit#8477
bojovyletoun wants to merge 1 commit into
roundcube:masterfrom
bojovyletoun:patch-1

Conversation

@bojovyletoun

@bojovyletoun bojovyletoun commented Mar 12, 2022

Copy link
Copy Markdown

[Original name "Allow to set HTML Content-Transfer-Encoding by "Use MIME for 8bit config setting]

I modified Content-Transfer-Encoding for HTML parts. Previously it was hardcoded quoted-printable. Now it is determined in same manner as for Text part.

Is there any caveat in it? I have Settings-Composing-Advanced:Use MIME for 8bit obviously off so it is effectively 8bit.

Allows to set Content-Transfer-Encoding for HTML part in similiar way as for Text part. (Based on Settings-Writing-Use MIME for 8bit)
If it is intentional (bugs in user agents...), please explain. According to my observations nearly everyone uses it and I think I should not be problematic, but I am not sure totally.

… "Use MIME for 8bit" setting

Allows to set Content-Transfer-Encoding for HTML part in similiar way as for Text part. (Based on Settings-Writing-Use MIME for 8bit) Previously it was hardcoded quoted-printable.
If it is intentional (bugs in user agents...), please explain. According to my observations nearly everyone uses it and I think I should not be problematic, but I am not sure totally.
@alecpl

alecpl commented May 28, 2023

Copy link
Copy Markdown
Member

A slightly modified version pushed.

@alecpl alecpl closed this May 28, 2023
@Windkracht8

Copy link
Copy Markdown

Hi, I've had to revert this in my installation because Mail_mime does not wrap lines when html_encoding is not quoted-printable or base64. My Exim server rejects emails where the lines are too long, they should not be over 998 octets.
8bit is not necessarily an issue, it just means Mail_mime won't wrap the lines and this needs to be done somewhere else.

alecpl added a commit that referenced this pull request May 31, 2026
alecpl added a commit that referenced this pull request May 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants