Merge pull request #26762 from nextcloud/bugfix/noid/respect-the-error-level-on-logging
Respect the error level when logging
This commit is contained in:
commit
303d689001
|
@ -90,13 +90,30 @@ class ErrorHandler {
|
||||||
}
|
}
|
||||||
$msg = $message . ' at ' . $file . '#' . $line;
|
$msg = $message . ' at ' . $file . '#' . $line;
|
||||||
$e = new \Error(self::removePassword($msg));
|
$e = new \Error(self::removePassword($msg));
|
||||||
self::$logger->logException($e, ['app' => 'PHP']);
|
self::$logger->logException($e, ['app' => 'PHP', 'level' => self::errnoToLogLevel($number)]);
|
||||||
}
|
}
|
||||||
|
|
||||||
//Recoverable handler which catch all errors, warnings and notices
|
//Recoverable handler which catch all errors, warnings and notices
|
||||||
public static function onAll($number, $message, $file, $line) {
|
public static function onAll($number, $message, $file, $line) {
|
||||||
$msg = $message . ' at ' . $file . '#' . $line;
|
$msg = $message . ' at ' . $file . '#' . $line;
|
||||||
$e = new \Error(self::removePassword($msg));
|
$e = new \Error(self::removePassword($msg));
|
||||||
self::$logger->logException($e, ['app' => 'PHP', 'level' => 0]);
|
self::$logger->logException($e, ['app' => 'PHP', 'level' => self::errnoToLogLevel($number)]);
|
||||||
|
}
|
||||||
|
|
||||||
|
public static function errnoToLogLevel(int $errno): int {
|
||||||
|
switch ($errno) {
|
||||||
|
case E_USER_WARNING:
|
||||||
|
return ILogger::WARN;
|
||||||
|
|
||||||
|
case E_USER_DEPRECATED:
|
||||||
|
return ILogger::DEBUG;
|
||||||
|
|
||||||
|
case E_USER_NOTICE:
|
||||||
|
return ILogger::INFO;
|
||||||
|
|
||||||
|
case E_USER_ERROR:
|
||||||
|
default:
|
||||||
|
return ILogger::ERROR;
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue