Fix debug log censoring collisions (EXPOSUREAPP-7196) (#3230)
* Detect censoring collision and replace whole line with placeholder. Give the debug logger an additional thread to parallelize censoring. * Smart censoring collision handling. * Tweak logging performance. * Adjusted unit tests to censoring changes. * Print stacktraces to debug log too! * Make KLINT happy * Adjust tests to changed log formatting. * Add tests for throwable formatting in debuglogger lines. * Re-enable debug tree. * Fix incorrect censoring range. We need the range of the original match, not it's replacement. * Fix duplicate new line if exceptions are logged. Co-authored-by:Kolya Opahle <k.opahle@sap.com> Co-authored-by:
harambasicluka <64483219+harambasicluka@users.noreply.github.com> Co-authored-by:
Mohamed Metwalli <mohamed.metwalli@sap.com>
Showing
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/bugreporting/censors/BugCensor.kt 36 additions, 4 deletions...va/de/rki/coronawarnapp/bugreporting/censors/BugCensor.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/bugreporting/censors/contactdiary/DiaryEncounterCensor.kt 8 additions, 6 deletions...bugreporting/censors/contactdiary/DiaryEncounterCensor.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/bugreporting/censors/contactdiary/DiaryLocationCensor.kt 10 additions, 8 deletions.../bugreporting/censors/contactdiary/DiaryLocationCensor.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/bugreporting/censors/contactdiary/DiaryPersonCensor.kt 10 additions, 8 deletions...pp/bugreporting/censors/contactdiary/DiaryPersonCensor.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/bugreporting/censors/contactdiary/DiaryVisitCensor.kt 8 additions, 6 deletions...app/bugreporting/censors/contactdiary/DiaryVisitCensor.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/bugreporting/censors/presencetracing/CheckInsCensor.kt 9 additions, 7 deletions...pp/bugreporting/censors/presencetracing/CheckInsCensor.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/bugreporting/censors/presencetracing/TraceLocationCensor.kt 13 additions, 11 deletions...greporting/censors/presencetracing/TraceLocationCensor.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/bugreporting/censors/submission/CoronaTestCensor.kt 34 additions, 14 deletions...rnapp/bugreporting/censors/submission/CoronaTestCensor.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/bugreporting/censors/submission/PcrQrCodeCensor.kt 5 additions, 8 deletions...arnapp/bugreporting/censors/submission/PcrQrCodeCensor.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/bugreporting/censors/submission/RACoronaTestCensor.kt 10 additions, 8 deletions...app/bugreporting/censors/submission/RACoronaTestCensor.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/bugreporting/censors/submission/RatProfileCensor.kt 18 additions, 13 deletions...rnapp/bugreporting/censors/submission/RatProfileCensor.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/bugreporting/censors/submission/RatQrCodeCensor.kt 12 additions, 10 deletions...arnapp/bugreporting/censors/submission/RatQrCodeCensor.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/bugreporting/censors/vaccination/CertificateQrCodeCensor.kt 28 additions, 29 deletions...greporting/censors/vaccination/CertificateQrCodeCensor.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/bugreporting/debuglog/DebugLogger.kt 25 additions, 3 deletions...de/rki/coronawarnapp/bugreporting/debuglog/DebugLogger.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/bugreporting/debuglog/LogLine.kt 28 additions, 8 deletions...ava/de/rki/coronawarnapp/bugreporting/debuglog/LogLine.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/bugreporting/debuglog/internal/DebugLogStorageCheck.kt 5 additions, 5 deletions...pp/bugreporting/debuglog/internal/DebugLogStorageCheck.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/bugreporting/debuglog/internal/DebugLoggerScope.kt 2 additions, 1 deletion...arnapp/bugreporting/debuglog/internal/DebugLoggerScope.kt
- Corona-Warn-App/src/main/java/de/rki/coronawarnapp/bugreporting/debuglog/internal/LogWriter.kt 7 additions, 6 deletions...coronawarnapp/bugreporting/debuglog/internal/LogWriter.kt
- Corona-Warn-App/src/test/java/de/rki/coronawarnapp/bugreporting/censors/BugCensorTest.kt 4 additions, 12 deletions...e/rki/coronawarnapp/bugreporting/censors/BugCensorTest.kt
- Corona-Warn-App/src/test/java/de/rki/coronawarnapp/bugreporting/censors/CheckInsCensorTest.kt 15 additions, 40 deletions.../coronawarnapp/bugreporting/censors/CheckInsCensorTest.kt
Loading
Please register or sign in to comment