diff options
| author | Morris Jobke <hey@morrisjobke.de> | 2015-09-24 17:01:31 +0200 |
|---|---|---|
| committer | Morris Jobke <hey@morrisjobke.de> | 2015-09-24 20:01:30 +0200 |
| commit | db8e7ce8b95c882c876f932296f25ec08883a1d3 (patch) | |
| tree | 807952698b4a0704172285f989563ac572df2762 /lib/private/log.php | |
| parent | 846b8268672df2c626616fec5f71e15570005e84 (diff) | |
| download | nextcloud-server-db8e7ce8b95c882c876f932296f25ec08883a1d3.tar.gz nextcloud-server-db8e7ce8b95c882c876f932296f25ec08883a1d3.zip | |
Remove passwords from logged exception stack traces
* fixed #16318
* create logException in ILogger
* add unit tests
Diffstat (limited to 'lib/private/log.php')
| -rw-r--r-- | lib/private/log.php | 21 |
1 files changed, 21 insertions, 0 deletions
diff --git a/lib/private/log.php b/lib/private/log.php index 3c0e7b45d1b..1b26ece7624 100644 --- a/lib/private/log.php +++ b/lib/private/log.php @@ -241,4 +241,25 @@ class Log implements ILogger { call_user_func(array($logger, 'write'), $app, $message, $level); } } + + /** + * Logs an exception very detailed + * + * @param \Exception $exception + * @param array $context + * @return void + * @since 8.2.0 + */ + public function logException(\Exception $exception, array $context = array()) { + $exception = array( + 'Exception' => get_class($exception), + 'Message' => $exception->getMessage(), + 'Code' => $exception->getCode(), + 'Trace' => $exception->getTraceAsString(), + 'File' => $exception->getFile(), + 'Line' => $exception->getLine(), + ); + $exception['Trace'] = preg_replace('!(login|checkPassword)\(.*\)!', '$1(*** username and password replaced ***)', $exception['Trace']); + $this->error('Exception: ' . json_encode($exception), $context); + } } |