[Mailer][Mime] In some areas header is not treated in a case-insensitive manner #39953
Labels
Comments
|
Unfortunately I can't :( |
fabpot
added a commit
that referenced
this issue
Feb 2, 2021
This PR was merged into the 4.4 branch. Discussion ---------- [Mime] Fix case-sensitive handling of header names | Q | A | ------------- | --- | Branch? | 4.4 | Bug fix? | yes | New feature? | no | Deprecations? | no | Tickets | Fix #39953 | License | MIT | Doc PR | - Fixes case-sensitive handling of header names in "Mailer" component, more in the [ticket](#39953) and the [root PR](#39954). Commits ------- b2d7454 [Mime] Fix case-sensitive handling in Headers::isUniqueHeader()
fabpot
added a commit
that referenced
this issue
Feb 2, 2021
… (piku235) This PR was squashed before being merged into the 5.2 branch. Discussion ---------- [Mailer][Mime] Fix case-sensitive handling of header names | Q | A | ------------- | --- | Branch? | 5.2 | Bug fix? | yes | New feature? | no | Deprecations? | no | Tickets | Fix #39953 | License | MIT | Doc PR | - Fixes case-sensitive handling of header names in "Mime" and "Mailer" component, more in the [ticket](#39953). Commits ------- d563c84 [Mailer][Mime] Fix case-sensitive handling of header names
symfony-splitter
pushed a commit
to symfony/mime
that referenced
this issue
Feb 2, 2021
… (piku235) This PR was squashed before being merged into the 5.2 branch. Discussion ---------- [Mailer][Mime] Fix case-sensitive handling of header names | Q | A | ------------- | --- | Branch? | 5.2 | Bug fix? | yes | New feature? | no | Deprecations? | no | Tickets | Fix #39953 | License | MIT | Doc PR | - Fixes case-sensitive handling of header names in "Mime" and "Mailer" component, more in the [ticket](symfony/symfony#39953). Commits ------- d563c846f6 [Mailer][Mime] Fix case-sensitive handling of header names
symfony-splitter
pushed a commit
to symfony/mailer
that referenced
this issue
Feb 2, 2021
… (piku235) This PR was squashed before being merged into the 5.2 branch. Discussion ---------- [Mailer][Mime] Fix case-sensitive handling of header names | Q | A | ------------- | --- | Branch? | 5.2 | Bug fix? | yes | New feature? | no | Deprecations? | no | Tickets | Fix #39953 | License | MIT | Doc PR | - Fixes case-sensitive handling of header names in "Mime" and "Mailer" component, more in the [ticket](symfony/symfony#39953). Commits ------- d563c846f6 [Mailer][Mime] Fix case-sensitive handling of header names
symfony-splitter
pushed a commit
to symfony/mailer
that referenced
this issue
Sep 28, 2021
… (piku235) This PR was squashed before being merged into the 5.2 branch. Discussion ---------- [Mailer][Mime] Fix case-sensitive handling of header names | Q | A | ------------- | --- | Branch? | 5.2 | Bug fix? | yes | New feature? | no | Deprecations? | no | Tickets | Fix #39953 | License | MIT | Doc PR | - Fixes case-sensitive handling of header names in "Mime" and "Mailer" component, more in the [ticket](symfony/symfony#39953). Commits ------- d563c846f6 [Mailer][Mime] Fix case-sensitive handling of header names
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Symfony version(s) affected: 5.2
Description
Recently, I've had a problem with a capitalized header when I tried to define a global sender in the framework config. The issue happens in the
Headers::addHeader()when trying to resolve a header class from the header class map.Also,
Headers::isUniqueHeader()is affected and header rules in theMessageListener::addHeaderRule().The feature pull request for reference.
How to reproduce
The following config leads to an exception in
Headers::addHeader()Possible Solution
The fix is simple, we need to ensure that
strtolower()is used when doing something specific on a user-supplied header name.The text was updated successfully, but these errors were encountered: