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

[Monolog] Mailer sends second mail if monolog is configured to email errors #17217

Closed
stollr opened this issue Aug 22, 2022 · 6 comments
Closed

Comments

@stollr
Copy link
Contributor

stollr commented Aug 22, 2022

As pointed out here there is an issue with the configuration example on https://symfony.com/doc/5.4/logging/monolog_email.html (same issue with newer versions of the doc). Following the first config block, one will always get two error mails if an error occurs.

@raziel057 already explained the reason for the second mail and how to fix it.

I suggest to add the channel constraint to the main handler to the documentation example. So that we have a working example.

What do you think?

@94noni
Copy link
Contributor

94noni commented Aug 23, 2022

and how to fix it.

To me it is more how to hide it
Excluding mailer channel can hide other valueable logs i think

But i do not know how to properly fix the root issue, appart removing the log :/

@stollr
Copy link
Contributor Author

stollr commented Aug 25, 2022

Excluding mailer channel can hide other valueable logs i think

Yes, that was my initial thought, too. But if the mail was sent, there is at least no fatal issue with the mail sending ;-)

@carsonbot
Copy link
Collaborator

Thank you for this issue.
There has not been a lot of activity here for a while. Has this been resolved?

@javiereguiluz
Copy link
Member

I don't know if this is a doc issue or a code issue in Monolog bundle. See symfony/monolog-bundle#405

What should we do here?

@javiereguiluz
Copy link
Member

Let's close for now in favor of symfony/monolog-bundle#405 and we'll reopen if needed when a decision is done there. Thanks.

@stollr
Copy link
Contributor Author

stollr commented Sep 18, 2023

That's fine for me. Thanks 👍

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

No branches or pull requests

4 participants