Improved crash logging & handling (EXPOSUREAPP-2572) (#1475)
* FileLoggerPlus - RX added - preparation * Buffer config as constants * avoid empty lines * Reworked logging to file - wip * Collecting errors to dao - wip * Added crash report to app db and created module * do it * inject crashReportTree - wip * use dagger injection * retrieve short hash via gradle * removed todo * Updated database to version 2 for new ReportEntity - old base * deprecated old engagement - RX not allowed anymore * use global scope for coroutines in crash report tree - ktlint clean * added menu item - error reporter (replace later to new structure) * basic recylcerview and fragment added to menu - wip * deprecated data class * Delete CrashModel.kt * crash report list item to use data binding * separated Reporteradapter - klint clean * use separated Adapter in ReporterFragment - cleanup * Added ErrorReporterViewModel - connected data to list * dependency injection for crash report, better structure, todo cleanup * some refactoring, cleanup, ktlint * adjustments for 1.6.x * temp action to open crash report fragment * removed testcode * updated layout - wip * connected view & data * satisfy circle ci checks * plant reportingTree, testbutton, klint clean * updated layout for reporter - wip * better format and layout * better message handling * missing icons for errors on debug * Adjusted error report index * Removed ugly show button, report clickable * Added SettingsCrashReportDetailsFragment & corresponding files * fixed merge inconsistence * fixed injection overall * use livedata * details added to report - wip * beauty * share & layout root preparation * make error report shareable * removed Dispatcher.IO for example * Introduce skeleton classes for bugreporting * Refactoring, adjustments to be able to build again * Created BugEventEntity, Changed id to Long for more efficiency, Removed unused events * refactored ui to deviceForTesters, testmenu etc * adjusted interfaces & related, better structure * uses own db, added converter, adjusted modules * adjusted bugrepo, defaultbugrepo * shared report, better visualisation * changed crash db name * Implemented DefaultBugRepository * Added device info to BugEvent, BugReporter collects Tag, Added default impl of BugEvent etc * corrected bug in RollingLogHistory * removed deprecated crash package * quickfix - wip * cleanup * cleanUp, adjusted simulation button to new structure * moved bug related layouts to deviceForTesters package * removed old RX dependenies and related context * fix build for deviceDebug * lint clean * current set of relevant exceptions as before * attach log to output * improved output * Delete obsolet ic_bug in main * Delete ic_coffee.xml * changed shortID to shortCommitHash for better transparency * companion to bottom * removed string encapsulation * cleanup unused import and blank line - PR Comment * typo fixed * rm empty line * removed relict module * layout fix * adjusted unrelated method call * fixed package inkonsistence * Use flow instead of livedata in non-ui * better fun naming * Replaced interface with inline listener * Improved SettingsCrashReportDetailsFragment and SettingsCrashReportViewModel, formatting adjusted * cleaned converter * Delete Project.xml * added log statement to DefaultBugRepository * adjusted magical number * Use CommonConverters instead of unique Converters in BugDatabase * Extension function for the resuce -defaultBugRepository * removed FormatterCrashReport completly * changed long to UUID * satisfy buggy ktlint checks * Update SettingsCrashReportViewModel.kt use postvalue * avoid passing viewmodel * store appVersionCode as Long for beeing futuresave * Update SettingsCrashReportViewModel.kt removed lazy init * Changed BugDatabase to factory pattern Co-authored-by:Matthias Urhahn <matthias.urhahn@sap.com> Co-authored-by:
harambasicluka <64483219+harambasicluka@users.noreply.github.com>
Showing
- .idea/codeStyles/Project.xml 0 additions, 136 deletions.idea/codeStyles/Project.xml
- Corona-Warn-App/build.gradle 12 additions, 0 deletionsCorona-Warn-App/build.gradle
- Corona-Warn-App/schemas/de.rki.coronawarnapp.bugreporting.storage.BugDatabase/1.json 112 additions, 0 deletions...rki.coronawarnapp.bugreporting.storage.BugDatabase/1.json
- Corona-Warn-App/schemas/de.rki.coronawarnapp.storage.AppDatabase/2.json 231 additions, 0 deletions...p/schemas/de.rki.coronawarnapp.storage.AppDatabase/2.json
- Corona-Warn-App/src/device/java/de/rki/coronawarnapp/bugreporting/BugReportingModule.kt 28 additions, 0 deletions...a/de/rki/coronawarnapp/bugreporting/BugReportingModule.kt
- Corona-Warn-App/src/deviceForTesters/java/de/rki/coronawarnapp/bugreporting/BugReportingModule.kt 41 additions, 0 deletions...a/de/rki/coronawarnapp/bugreporting/BugReportingModule.kt
- Corona-Warn-App/src/deviceForTesters/java/de/rki/coronawarnapp/bugreporting/event/BugEvent.kt 21 additions, 0 deletions.../java/de/rki/coronawarnapp/bugreporting/event/BugEvent.kt
- Corona-Warn-App/src/deviceForTesters/java/de/rki/coronawarnapp/bugreporting/event/BugEventEntity.kt 24 additions, 0 deletions...de/rki/coronawarnapp/bugreporting/event/BugEventEntity.kt
- Corona-Warn-App/src/deviceForTesters/java/de/rki/coronawarnapp/bugreporting/event/DefaultBugEvent.kt 21 additions, 0 deletions...e/rki/coronawarnapp/bugreporting/event/DefaultBugEvent.kt
- Corona-Warn-App/src/deviceForTesters/java/de/rki/coronawarnapp/bugreporting/loghistory/RollingLogHistory.kt 64 additions, 0 deletions...oronawarnapp/bugreporting/loghistory/RollingLogHistory.kt
- Corona-Warn-App/src/deviceForTesters/java/de/rki/coronawarnapp/bugreporting/processor/BugProcessor.kt 7 additions, 0 deletions.../rki/coronawarnapp/bugreporting/processor/BugProcessor.kt
- Corona-Warn-App/src/deviceForTesters/java/de/rki/coronawarnapp/bugreporting/processor/DefaultBugProcessor.kt 52 additions, 0 deletions...ronawarnapp/bugreporting/processor/DefaultBugProcessor.kt
- Corona-Warn-App/src/deviceForTesters/java/de/rki/coronawarnapp/bugreporting/reporter/DefaultBugReporter.kt 29 additions, 0 deletions...coronawarnapp/bugreporting/reporter/DefaultBugReporter.kt
- Corona-Warn-App/src/deviceForTesters/java/de/rki/coronawarnapp/bugreporting/storage/BugDatabase.kt 33 additions, 0 deletions.../de/rki/coronawarnapp/bugreporting/storage/BugDatabase.kt
- Corona-Warn-App/src/deviceForTesters/java/de/rki/coronawarnapp/bugreporting/storage/dao/BugDao.kt 12 additions, 0 deletions...a/de/rki/coronawarnapp/bugreporting/storage/dao/BugDao.kt
- Corona-Warn-App/src/deviceForTesters/java/de/rki/coronawarnapp/bugreporting/storage/dao/DefaultBugDao.kt 26 additions, 0 deletions...i/coronawarnapp/bugreporting/storage/dao/DefaultBugDao.kt
- Corona-Warn-App/src/deviceForTesters/java/de/rki/coronawarnapp/bugreporting/storage/repository/BugRepository.kt 11 additions, 0 deletions...awarnapp/bugreporting/storage/repository/BugRepository.kt
- Corona-Warn-App/src/deviceForTesters/java/de/rki/coronawarnapp/bugreporting/storage/repository/DefaultBugRepository.kt 50 additions, 0 deletions...p/bugreporting/storage/repository/DefaultBugRepository.kt
- Corona-Warn-App/src/deviceForTesters/java/de/rki/coronawarnapp/test/DeviceForTestersModule.kt 3 additions, 1 deletion.../java/de/rki/coronawarnapp/test/DeviceForTestersModule.kt
- Corona-Warn-App/src/deviceForTesters/java/de/rki/coronawarnapp/test/crash.ui/CrashReportAdapter.kt 47 additions, 0 deletions.../de/rki/coronawarnapp/test/crash.ui/CrashReportAdapter.kt
Loading
Please register or sign in to comment