Skip to content

fix: try to avoid lines longer than 78 characters#32

Merged
mdecimus merged 1 commit intostalwartlabs:mainfrom
link2xt:link2xt/78-chars
Feb 9, 2025
Merged

fix: try to avoid lines longer than 78 characters#32
mdecimus merged 1 commit intostalwartlabs:mainfrom
link2xt:link2xt/78-chars

Conversation

@link2xt
Copy link
Contributor

@link2xt link2xt commented Feb 7, 2025

https://www.rfc-editor.org/rfc/rfc5322#section-2.1.1 says lines MUST not exceed 998 characters,
but SHOULD not exceed 78 characters.

Closes #31

@link2xt
Copy link
Contributor Author

link2xt commented Feb 7, 2025

There seems to be also a problem if the line does not contain \n at all, but maybe it is impossible in the code paths? If I just put a very long line into detect_encoding without \r\n, it does nothing.

cc @mdecimus

<https://www.rfc-editor.org/rfc/rfc5322#section-2.1.1>
says lines MUST not exceed 998 characters,
but SHOULD not exceed 78 characters.
@mdecimus mdecimus merged commit 748f49c into stalwartlabs:main Feb 9, 2025
1 check passed
@link2xt link2xt mentioned this pull request Feb 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Limit line length to 78 characters instead of 998

2 participants