8000 [Mime] use address for body at `PathHeader` by tinect · Pull Request #59796 · symfony/symfony · GitHub
[go: up one dir, main page]
Skip to content
/ symfony Public < 8000 /div>

[Mime] use address for body at PathHeader #59796

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Feb 19, 2025
Merged

[Mime] use address for body at PathHeader #59796

merged 1 commit into from
Feb 19, 2025

Conversation

tinect
Copy link
Contributor
@tinect tinect commented Feb 17, 2025 •
edited by OskarStark
Loading
Q A
Branch? 6.4
Bug fix? yes
New feature? no
Deprecations? no
Issues Fix #...
License MIT

We noticed, that ReturnPath has wrong content in EML if any library is forcing key=>value.
Setting it to mail@example.com, results into <"mail@example.com" <mail@example.com>> after stringify it which is not compliant: Email \"\"mail@example.com"<mail@example.com\" does not comply with addr-spec of RFC 2822.

@carsonbot carsonbot added this to the 6.4 milestone Feb 17, 2025
@carsonbot carsonbot changed the title [MIME] use address for body at PathHeader [Mime] use address for body at PathHeader Feb 17, 2025
@OskarStark OskarStark changed the title [Mime] use address for body at PathHeader [Mime] use address for body at PathHeader Feb 18, 2025
@symfony symfony deleted a comment from carsonbot Feb 18, 2025
@nicolas-grekas
Copy link
Member

Thank you @tinect.

@nicolas-grekas nicolas-grekas merged commit 757fcbb into symfony:6.4 Feb 19, 2025
10 of 11 checks passed
@tinect tinect deleted the returnPath branch February 19, 2025 09:37
This was referenced Feb 26, 2025
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Bug Mime Status: Needs Review
Projects
None yet
Milestone
6.4
Development

Successfully merging this pull request may close these issues.
3 participants
@tinect @nicolas-grekas @carsonbot
0