Skip to content

Commit 22c69f3

Browse files
#71: do not log an api error message when ignoring messages below parse level
1 parent 2f1d086 commit 22c69f3

File tree

3 files changed

+13
-2
lines changed

3 files changed

+13
-2
lines changed

src/BelowParseLevelException.php

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
<?php
2+
3+
namespace Rollbar\Laravel;
4+
5+
class BelowParseLevelException extends \Exception
6+
{
7+
}

src/RollbarLogHandler.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,7 @@ public function log($level, $message, array $context = [])
7070
{
7171
// Check if we want to log this message.
7272
if ($this->parseLevel($level) < $this->level) {
73-
return;
73+
throw new BelowParseLevelException();
7474
}
7575

7676
$context = $this->addContext($context);

src/RollbarServiceProvider.php

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,11 @@ public function boot()
4848
return;
4949
}
5050

51-
$result = $app[RollbarLogHandler::class]->log($level, $message, $context);
51+
try {
52+
$result = $app[RollbarLogHandler::class]->log($level, $message, $context);
53+
} catch(BelowParseLevelException $exception) {
54+
return;
55+
}
5256

5357
if (!$result || !$result->getStatus()) {
5458
\Log::error(

0 commit comments

Comments
 (0)