Movatterモバイル変換

Skip to content

symfony/symfony-docsPublic

NotificationsYou must be signed in to change notification settings
Fork5.3k
Star2.3k

[Mailer] Mention the`SentMessageEvent` and`FailedMessageEvent`#20186

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged

javiereguiluz merged 1 commit intosymfony:6.4fromwkania:mailer-debug-links-to-events

Feb 3, 2025

Merged

[Mailer] Mention the`SentMessageEvent` and`FailedMessageEvent`#20186

javiereguiluz merged 1 commit intosymfony:6.4fromwkania:mailer-debug-links-to-events

Feb 3, 2025

Conversation

@wkania

Copy link

Contributor

wkania commentedSep 1, 2024•
edited
Loading

Thesymfony/symfony#47080 PR was created to solve the problem ofdebugging and to getinformation about email after it was sent.
The section aboutdebugging 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:<example@symfony.com>< 250 2.1.0 Ok> RCPT TO:<dummy@symfony.com>< 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 Status: Needs Review Mailer labels

@carsonbot

carsonbot added this to the6.4 milestone

@wkania

wkania force-pushed themailer-debug-links-to-events branch from69e0ddd to4eb9f97Compare

September 1, 2024 22:21

@OskarStark

OskarStark changed the title~~[Mailer] Mention the SentMessageEvent and FailedMessageEvent in the d…~~[Mailer] Mention theSentMessageEvent andFailedMessageEvent

@wkania

wkania mentioned this pull request

Fix typo in mailer.rst#20199

Merged

@javiereguiluz

javiereguiluz added Status: Reviewed and removed Status: Needs Review labels

@wkania

@javiereguiluz

[Mailer] Mention the SentMessageEvent and FailedMessageEvent in the d…

8e3c1db

…ebug section

@javiereguiluz

javiereguiluz force-pushed themailer-debug-links-to-events branch from4eb9f97 to8e3c1dbCompare

February 3, 2025 11:52

@javiereguiluz

javiereguiluz merged commit0ea7ad3 intosymfony:6.4

3 checks passed

@javiereguiluz

Copy link

Member

javiereguiluz commentedFeb 3, 2025•
edited
Loading

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

@wkania

Copy link

ContributorAuthor

wkania commentedFeb 4, 2025

I'm glad it was merged.

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Labels

Mailer Status: Reviewed

3 participants

@wkania

@javiereguiluz

@carsonbot

[8]ページ先頭

©2009-2025 Movatter.jp