search

codeinternetapplications / monolog-stackdriver

Stackdriver handler for Monolog.
MIT License
15 stars 15 forks source link

StackdriverHandler does not comply with Monolog AbstractHandler #5

Closed kswzr closed 4 years ago

kswzr commented 4 years ago

Hey, using your implementation with monolog/monolog >=2.0 currently does not working, because StackdriverHandler does not comply with Monolog/Handler/AbstractHandler.

The return value of StackdriverHandler->write() should be void. The return value of StackdriverHandler->write() should be array.

Additionally we need to restrict the monolog version in the current releases to monolog/monolog: 1.*