Skip to content

Message->setBody: Clear headers when adding multi-part body#268

Open
vlk-charles wants to merge 1 commit intolaminas:2.26.xfrom
vlk-charles:issue265
Open

Message->setBody: Clear headers when adding multi-part body#268
vlk-charles wants to merge 1 commit intolaminas:2.26.xfrom
vlk-charles:issue265

Conversation

@vlk-charles
Copy link
Copy Markdown

It is undesirable for the Content-Type and Content-Transfer-Encoding headers to remain from a previous single-part call. Solves #265

It is undesirable for these headers to remain from a previous single-part call. Solves laminas#265

Signed-off-by: Karel Vlk <vlk.charles@gmail.com>
@arhimede
Copy link
Copy Markdown
Member

arhimede commented Sep 3, 2024

This library is discontinued.
You should switch to some other libraries in order to handle emails.

@vlk-charles
Copy link
Copy Markdown
Author

Thank you for the information. Is there an official statement available somewhere? I can't find any deprecation notice here or anywhere else.

Unfortunately I encountered this bug becasue a third-party project (iTop) recently switched to this library.

@arhimede
Copy link
Copy Markdown
Member

arhimede commented Sep 3, 2024

Thank you for the information. Is there an official statement available somewhere? I can't find any deprecation notice here or anywhere else.

Unfortunately I encountered this bug becasue a third-party project (iTop) recently switched to this library.

See below the status of all packages and libraries
https://getlaminas.org/packages-maintenance-status/

@froschdesign
Copy link
Copy Markdown
Member

@Slamdunk
It is your decision to accept or close the pull request.

@vlk-charles
Copy link
Copy Markdown
Author

Sorry about breaking the one Psalm test. I did some non-essential refactoring for (my) readability. I can fix it if necessary for acceptance of the pull request. But no point in committing new code if it won't be accepted for EOL reasons anyway.

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.

4 participants