Skip to content

Conversation

cmb69
Copy link
Member

@cmb69 cmb69 commented Nov 10, 2020

\r\n does only terminate a header, if not followed by \t or .
We have to cater to that when determining the end position of the
respective headers.

`\r\n` does only terminate a header, if not followed by `\t` or ` `.
We have to cater to that when determining the end position of the
respective headers.
@cmb69 cmb69 added the Bug label Nov 10, 2020
@Girgias
Copy link
Member

Girgias commented Nov 10, 2020

Does that only affect the base mail function or also IMAP?

@cmb69
Copy link
Member Author

cmb69 commented Nov 10, 2020

That patch should affect both functions (regarding the $additional_headers) for SMTP sending (sendmail is not affected).

@cmb69
Copy link
Member Author

cmb69 commented Nov 17, 2020

Any objections to merge this?

@php-pulls php-pulls closed this in 881e43a Nov 19, 2020
@cmb69 cmb69 deleted the cmb/72964 branch November 19, 2020 18:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants