Skip to content
This repository was archived by the owner on Nov 27, 2020. It is now read-only.

[Log] Wire server_log handler #1057

Merged
merged 1 commit into from
Apr 3, 2017
Merged

Conversation

lyrixx
Copy link
Member

@lyrixx lyrixx commented Mar 24, 2017

This is the last part of the new server:log command.

Maybe a release of the monolog bundle could be better to avoid to require a dev package.
ping @Seldaek

WDYT?

@Seldaek
Copy link
Member

Seldaek commented Mar 26, 2017

monologbundle 3.1.0 is out

@@ -22,6 +22,12 @@ monolog:
type: console
process_psr_3_messages: false
channels: ['!event', '!doctrine', '!console']
# To follow logs in real time, execute the following command:
# `bin/console server:log -vv`
Copy link
Member

Choose a reason for hiding this comment

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

Can we remove the need for -vv?

Copy link
Member Author

Choose a reason for hiding this comment

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

Actually, this is not mandatory ... it's just to get more logs.

Copy link
Member

Choose a reason for hiding this comment

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

Ah, I see, you only see warnings if excluded.

@lyrixx lyrixx force-pushed the server-log-enable branch from 947747f to 53396e4 Compare April 3, 2017 14:15
@lyrixx
Copy link
Member Author

lyrixx commented Apr 3, 2017

I rebased the PR. Now it required the new symfony/monolog-bundle package.

@fabpot
Copy link
Member

fabpot commented Apr 3, 2017

Thank you @lyrixx.

@fabpot fabpot merged commit 53396e4 into symfony:master Apr 3, 2017
fabpot added a commit that referenced this pull request Apr 3, 2017
This PR was merged into the 3.3-dev branch.

Discussion
----------

[Log] Wire server_log handler

This is the last part of the new `server:log` command.

Maybe a release of the monolog bundle could be better to avoid to require a dev package.
ping @Seldaek

WDYT?

Commits
-------

53396e4 [Log] Wire server_log handler
@lyrixx lyrixx deleted the server-log-enable branch April 4, 2017 07:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants