Exceptions thrown when reached error notification limit
Closed this issue · 1 comments
subzero10 commented
- Honeybadger is trying to report all logs (info, warning) and is reaching the notification limit.
- Errors are being thrown because the limit has been reached.
subzero10 commented
- Honeybadger is trying to report all logs (info, warning) and is reaching the notification limit.
✅ This has been fixed with v2.14.1
- Errors are being thrown because the limit has been reached.
I can't create a reproducible example of this.
In all my tests, the code works as expected:
- Initiate an HB client:
$hb = new Honeybadger([
'api_key' => '123',
'report_data' => true,
]);
- Send an error to HB:
$hb->notify(new Exception('This is a test exception from PHP!'))
- The request will fail at this step:
$this->handleServiceException((new ServiceExceptionFactory($response))->make());
-
The
handleServiceException
will call the configurableservice_exception_handler
, which has default implementation to throw the exception. -
Throwing the exception will cause the global exception handler to be invoked, as shown here.
-
This handler will call
$honeybadger->notify()
, but execution won't go far because$this->shouldReport()
will returnfalse
since the exception is an instance of aServiceException
.
I will close this issue and we can re-open if we find a way to reproduce this (if an issue exists).