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 Upgrade? #78

Closed
bassrock opened this issue Apr 15, 2014 · 5 comments
Closed

Monolog Upgrade? #78

bassrock opened this issue Apr 15, 2014 · 5 comments

Comments

@bassrock
Copy link

Currently this bundle is at 1.6 of monolog while the monolog repo is currently at 1.8. 1.8 provides newer GELF support as well as a flurry of new handlers. Is there a change that this bundle will get upgraded soon?

@xabbuh
Copy link
Member

xabbuh commented Apr 15, 2014

~1.6 includes 1.8. So, MonologBundle is not the package keeping you at1.6.

@bassrock
Copy link
Author

@xabbuh it doesn't seem to the compaiilble with 1.8's gelf component though:

PHP Fatal error:  Class 'Gelf\MessagePublisher' not found in /vagrant/app/cache/dev/appDevDebugProjectContainer.php on line 3897

@Seldaek
Copy link
Member

Seldaek commented Apr 16, 2014

Yes I guess we need to build in support for the new Gelf classes if monolog 1.8+ is used. If you use the old gelf library it should work though even with monolog 1.8.

@Seldaek
Copy link
Member

Seldaek commented Apr 16, 2014

@bassrock can you try & confirm it works with latest ~2.5@dev of the bundle?

@bassrock
Copy link
Author

@Seldaek seems to work! Graylog actually reads my logs now before the new version of graylog was rejecting them.

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

No branches or pull requests

3 participants