Alessandro Lai
Alessandro Lai
Pinning #286, should be interesting to this feature.
This seems to be the not first time this is raised as an issue, see #369. We could probably leverage the logger like in #370.
> The issue with integrations is that in the current version it always wants an array and passing a callback isn't allowed despite what the error message says. This seems...
`kernel.build_dir` [has been added in 5.2](https://github.com/symfony/symfony/commit/e98fdc7073a3f0c4db08efe5571d7dce5f74d15d), we need to make that optional, depending on the SF version.
@ste93cry sent me this: https://github.com/getsentry/sentry-symfony/blob/master/src/DependencyInjection/SentryExtension.php#L88 We already have a safeguard for this situation, but it's probably not working in your case.
I thought that we were already handling this use case in the listener: https://github.com/getsentry/sentry-symfony/blob/32fce0e687afb835505ec759de9fe1225b9d4d94/src/EventListener/MessengerListener.php#L50-L53 In which case this isn't working for you?
The `ErrorListener` acts only in the context of a request, so it would be ok to add specific code to handle this case. We could also add an E2E case...
> * the `IgnoreErrorsIntegration`? Is it OK to treat `HandlerFailedException` as "generic enough" that everyone who has the `IgnoreErrorsIntegration` should get this filtered out? Nope. We can't make such an...
This should be a nice feature. In any case, it's already possible with the `transport_factory` option: you need to write a class that implements the `TransportFactoryInterface` and just returns the...
As of now, the Symfony client is used as default transport not just by this bundle, but by the whole PHP SDK. Hence, [that piece of code](https://github.com/getsentry/sentry-php/blob/ecbd09ea5d053a202cf773cb24ab28af820831bd/src/HttpClient/HttpClientFactory.php#L116) doesn't know if...