Alessandro Lai
Alessandro Lai
@B-Galati would you like to help here? Due to your work on https://github.com/B-Galati/monolog-sentry-handler, you're probably an expert on this approach...
Thanks for the input @tgalopin! Do you think that that approach is feasible in this bundle? It doesn't seems so to me, but I'm unclear on some inner workings of...
> I'm not sure why it wouldn't be feasible, I just did it in a bundle myself :) . Are there specific reasons blocking you to do it here? But...
> I would be happy to help on any subject so feel free to ping me for code review, issues, submitting a PR or helping on a PR. Sure! I'm...
@B-Galati I've merged #337 and committed 02e6902, so now master is 4.x only. IMHO now we need a couple of PRs: * one to add a full monolog Handler *...
Probably having it built-in would make it easier to maintain, but feel free to attribute your work if you want to port it inside!
Do you mean committing it into the `sentry/sentry-php` package? Yeah sure, that's good. We would have to lock against the first version with that in though. It's fine for me.
Reporting an overlooked issue from https://github.com/getsentry/sentry-symfony/issues/363#issuecomment-685391154: > Using only monolog wouldn't allow to ignore soft fail exceptions. Which is a pretty useful feature. This regards the Messenger specifically, but I...
Sorry to do this, but we can't hold back the 4.0 release with this issue too, it's too big of a change and we're already behind with #399 and #374....
I've just had a possible idea on how to roll out this feature without a major version bump. Let me explain: * we already have a `register_error_listener` (default `true`) *...