-
Notifications
You must be signed in to change notification settings - Fork 9.4k
Update \Magento\Email\Model\Transport::setReturnPath to use the exist… #39881
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: 2.4-develop
Are you sure you want to change the base?
Conversation
…ing logic defined in [Headers.php v6.4.19](https://github.com/symfony/mime/blob/v6.4.19/Header/Headers.php)
Hi @TomashKhamlai. Thank you for your contribution!
Allowed build names are:
You can find more information about the builds here For more details, review the Code Contributions documentation. |
@magento run Magento Health Index |
@magento run Static Tests |
@magento run all tests |
@magento run Unit Tests |
Failed to run the builds. Please try to re-run them later. |
@magento run Sample Data Tests CE |
Failed to run the builds. Please try to re-run them later. |
@magento run Sample Data Tests CE |
Failed to run the builds. Please try to re-run them later. |
Later, like in 2026?! @magento you are hilarious! Stop dragging your feet and get the tests done! |
@magento run all tests |
Description (*)
An issue was identified where the Sender header is not following RFC 2822 while Symfony does follow
Update
\Magento\Email\Model\Transport::setReturnPath
to use the existing logic defined in Headers.php v6.4.19Related Pull Requests
Fixed Issues (if relevant)
Manual testing scenarios (*)
Questions or comments
Contribution checklist (*)