Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Document spool type#2742

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

Already on GitHub?Sign in to your account

Closed
tvlooy wants to merge1 commit intosymfony:masterfromtvlooy:mail_tip
Closed

Document spool type#2742

tvlooy wants to merge1 commit intosymfony:masterfromtvlooy:mail_tip

Conversation

@tvlooy
Copy link
Contributor

This is a known thing (https://github.com/symfony/symfony-standard/issues/425) but not documented. I think many people will trip over this, so I'm suggesting adding a tip about it.

QA
Doc fix?yes
New docs?no
Applies toall
Fixed tickets

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

The filename should be in a literal too

@wouterj
Copy link
Member

@weaverryan it should be merged in the 2.2 branch

@weaverryan
Copy link
Member

I think this is a good note to add. BUT, I'm not sure if the note is correct yet:

  1. My impression from looking at symfony/symfony-standard#425 is that this has nothing to do with 500 errors. My understanding is that if my application spools emails in memory, and then an Exception is thrown, the terminate event would still be triggered and the spooled emails sent. What this issue seems to be is simply that if monolog has spooled any mails (which in production, it usually only logs 500 level things), those will not be sent. Is this correct? If so, I think we should change the message - it sounds like a 500-level error will prevent my application emails to not send either.

  2. The "You need to comment out the ... so the mails will be sent immediately" gave me the impression (which I know you don't intend) that the if my application fails to send some memory-spooled emails, that after I change this configuration, it will immediately "catch-up" and send all the emails that were previously-not sent. I know that's not what you're trying to say, so perhaps we can reword this as well :).

Thanks for taking this on - obviously a lot of people are coming across this issue, so I would like to cover it in the docs.

Cheers!

@weaverryan
Copy link
Member

Hi Tom!

This issue is now closed in#2830.

Thanks!

@tvlooy
Copy link
ContributorAuthor

Ah. Totally forgot about this one. Glad someone else fixed it! Thanks.

@tvlooytvlooy deleted the mail_tip branchDecember 15, 2013 18:40
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@tvlooy@wouterj@weaverryan

[8]ページ先頭

©2009-2025 Movatter.jp