simplesamlphp
simplesamlphp copied to clipboard
WIP: PSR-3 compatible Logger
This checks some of the boxes in #371:
- [x] Extract the log levels definition from \SimpleSAML\Logger and use the ones defined by Psr\Log\LogLevel instead.
- [x] Add a
$contextparameter to all the methods defined byPsr\Log\LoggerInterface. - [x] Make the
\SimpleSAML\Loggerclass extendPsr\Log\AbstractLogger. - [x] Make the
\SimpleSAML\Logger::log()method public and make it throw a\Psr\Log\InvalidArgumentExceptionif the log level is unknown.
Codecov Report
Merging #1438 (bfaeb14) into master (7a1d8e6) will decrease coverage by
6.96%. The diff coverage is33.33%.
:exclamation: Current head bfaeb14 differs from pull request most recent head aa21c42. Consider uploading reports for the commit aa21c42 to get more accurate results
Additional details and impacted files
@@ Coverage Diff @@
## master #1438 +/- ##
============================================
- Coverage 45.48% 38.53% -6.96%
+ Complexity 3692 3482 -210
============================================
Files 158 139 -19
Lines 12292 10373 -1919
============================================
- Hits 5591 3997 -1594
+ Misses 6701 6376 -325