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

MonologBundle: use excluded_http_codes #604

Merged

Conversation

Projects
None yet
3 participants
@weaverryan
Copy link
Member

commented Jun 2, 2019

Q A
License MIT

This uses excluded_http_codes instead of excluded_404s, which is more flexible (and excluded_404s is deprecated in the next, unreleased version of MonologBundle).

@symfony-flex-server
Copy link
Contributor

left a comment

Pull request passes validation.

type: fingers_crossed
action_level: error
handler: nested
excluded_http_codes: [404, 405]

This comment has been minimized.

Copy link
@weaverryan

weaverryan Jun 2, 2019

Author Member

This line is the only change:

-            excluded_404s:
-                # regex: exclude all 404 errors from the logs
-                - ^/
+            excluded_http_codes: [404, 405]
@fabpot

fabpot approved these changes Jun 3, 2019

@symfony-flex-server symfony-flex-server bot merged commit 59e5741 into symfony:master Jun 3, 2019

1 check passed

continuous-integration/symfony/pr Done
Details

symfony-flex-server bot added a commit that referenced this pull request Jun 3, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.