Skip to content
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

[Mailer] Mention the SentMessageEvent and FailedMessageEvent #20186

Merged
merged 1 commit into from
Feb 3, 2025

Conversation

wkania
Copy link
Contributor

@wkania wkania commented Sep 1, 2024

The symfony/symfony#47080 PR was created to solve the problem of debugging and to get information about email after it was sent.
The section about debugging does not mention those events. Added some links for better navigation.

For example what debug returns:

< 220 ESMTP SYMFONY.COM
> EHLO [127.0.0.1]
< 250-smtp.symfony.com
< 250-PIPELINING
< 250-SIZE 157286400
< 250-AUTH PLAIN LOGIN PLAIN LOGIN PLAIN LOGIN
< 250-AUTH=PLAIN LOGIN PLAIN LOGIN PLAIN LOGIN
< 250-ENHANCEDSTATUSCODES
< 250-8BITMIME
< 250 SMTPUTF8
> AUTH LOGIN
< 334 VXNlcm5hbWU6
> ZXhhbXBsZUBzeW1mb255LmNvbQ==
< 334 UGFzc3dvcmQ6
> U3ltcGhvbnkg.aXMgQXdlc29tZQ==
< 235 2.7.0 Authentication successful
> MAIL FROM:<[email protected]>
< 250 2.1.0 Ok
> RCPT TO:<[email protected]>
< 250 2.1.5 Ok
> DATA
< 354 End data with <CR><LF>.<CR><LF>
> .
< 250 OK. ID: a20fb6ebbc54d22b

P.S.
I see that the checks now can find repeated words:

mailer.rst ✘
 1765: The word "the" is used more times in a row.
   ->  ``FailedMessageEvent`` allows acting on the the initial message in case of a failure and some

@carsonbot carsonbot added this to the 6.4 milestone Sep 1, 2024
@wkania wkania force-pushed the mailer-debug-links-to-events branch from 69e0ddd to 4eb9f97 Compare September 1, 2024 22:21
@OskarStark OskarStark changed the title [Mailer] Mention the SentMessageEvent and FailedMessageEvent in the d… [Mailer] Mention the SentMessageEvent and FailedMessageEvent Sep 2, 2024
@wkania wkania mentioned this pull request Sep 5, 2024
@javiereguiluz javiereguiluz force-pushed the mailer-debug-links-to-events branch from 4eb9f97 to 8e3c1db Compare February 3, 2025 11:52
@javiereguiluz javiereguiluz merged commit 0ea7ad3 into symfony:6.4 Feb 3, 2025
3 checks passed
@javiereguiluz
Copy link
Member

javiereguiluz commented Feb 3, 2025

This was finally merged! Thanks a lot @wkania and sorry it took us so long to merge it.

@wkania
Copy link
Contributor Author

wkania commented Feb 4, 2025

I'm glad it was merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants