monolog-bundle
monolog-bundle copied to clipboard
fixes #342 add rfc option for syslogudp handler
Can you review latest changes?
Hi! When can we expect a merge? This very important for me. It seems like nothing else prevents the merge.
Also highly requested from my side, need it to select RFC5424e (time with miliseconds) for SyslogUDP logging.
It would be nice to have a test case in Tests/DependencyInjection/ConfigurationTest.php to see if the default is used when rfc is not set in config and another case for when an invalid number is picked.
Done
Can I expect a review soon? It seems to me important and necessary to contribute to open source projects, especially if the proposed changes are needed not only by one developer, but, frankly, it is a little confusing how quickly the library development team considers proposals, this PR was created a year ago
A last comment for my side ;-).
Don't forget to add an entry in the ChangeLog pray
done )