diff --git a/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/deadman/DeadmanNotificationOneTimeWorker.kt b/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/deadman/DeadmanNotificationOneTimeWorker.kt index 039330a1ca998c927852a16cf9d0a0465b072503..9eec045cc1776734cbad047b9c822b3db65cbe65 100644 --- a/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/deadman/DeadmanNotificationOneTimeWorker.kt +++ b/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/deadman/DeadmanNotificationOneTimeWorker.kt @@ -22,10 +22,10 @@ class DeadmanNotificationOneTimeWorker @AssistedInject constructor( CoroutineWorker(context, workerParams) { override suspend fun doWork(): Result { - Timber.d("Background job started. Run attempt: $runAttemptCount") + Timber.tag(TAG).d("Background job started. Run attempt: $runAttemptCount") if (runAttemptCount > BackgroundConstants.WORKER_RETRY_COUNT_THRESHOLD) { - Timber.d("Background job failed after $runAttemptCount attempts. Rescheduling") + Timber.tag(TAG).d("Background job failed after $runAttemptCount attempts. Rescheduling") return Result.failure() } @@ -41,4 +41,8 @@ class DeadmanNotificationOneTimeWorker @AssistedInject constructor( @AssistedInject.Factory interface Factory : InjectedWorkerFactory<DeadmanNotificationOneTimeWorker> + + companion object { + private val TAG = DeadmanNotificationOneTimeWorker::class.java.simpleName + } } diff --git a/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/deadman/DeadmanNotificationPeriodicWorker.kt b/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/deadman/DeadmanNotificationPeriodicWorker.kt index 53e5d06ca010c8e45f40be61ab9219f0141d0381..efd968e15ea379950f0a7b1dacc8e6a39d6ee66f 100644 --- a/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/deadman/DeadmanNotificationPeriodicWorker.kt +++ b/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/deadman/DeadmanNotificationPeriodicWorker.kt @@ -22,10 +22,10 @@ class DeadmanNotificationPeriodicWorker @AssistedInject constructor( CoroutineWorker(context, workerParams) { override suspend fun doWork(): Result { - Timber.d("Background job started. Run attempt: $runAttemptCount") + Timber.tag(TAG).d("Background job started. Run attempt: $runAttemptCount") if (runAttemptCount > BackgroundConstants.WORKER_RETRY_COUNT_THRESHOLD) { - Timber.d("Background job failed after $runAttemptCount attempts. Rescheduling") + Timber.tag(TAG).d("Background job failed after $runAttemptCount attempts. Rescheduling") return Result.failure() } @@ -34,7 +34,7 @@ class DeadmanNotificationPeriodicWorker @AssistedInject constructor( // Schedule one time deadman notification send work scheduler.scheduleOneTime() } catch (e: Exception) { - Timber.d(e) + Timber.tag(TAG).d(e) result = Result.retry() } @@ -43,4 +43,8 @@ class DeadmanNotificationPeriodicWorker @AssistedInject constructor( @AssistedInject.Factory interface Factory : InjectedWorkerFactory<DeadmanNotificationPeriodicWorker> + + companion object { + private val TAG = DeadmanNotificationPeriodicWorker::class.java.simpleName + } } diff --git a/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/nearby/ExposureStateUpdateWorker.kt b/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/nearby/ExposureStateUpdateWorker.kt index e383844c36934a71d25d72f588af93f36b11dae9..5332793bffb9cff398fb6bae06816085bd8a7086 100644 --- a/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/nearby/ExposureStateUpdateWorker.kt +++ b/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/nearby/ExposureStateUpdateWorker.kt @@ -22,12 +22,10 @@ class ExposureStateUpdateWorker @AssistedInject constructor( override suspend fun doWork(): Result { try { - Timber.v("worker to persist exposure summary started") - taskController.submit( DefaultTaskRequest(RiskLevelTask::class, originTag = "ExposureStateUpdateWorker") ) - Timber.v("risk level calculation triggered") + Timber.tag(TAG).v("Risk level calculation triggered") } catch (e: ApiException) { e.report(ExceptionCategory.EXPOSURENOTIFICATION) } @@ -37,4 +35,8 @@ class ExposureStateUpdateWorker @AssistedInject constructor( @AssistedInject.Factory interface Factory : InjectedWorkerFactory<ExposureStateUpdateWorker> + + companion object { + private val TAG = ExposureStateUpdateWorker::class.java.simpleName + } } diff --git a/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/worker/BackgroundNoiseOneTimeWorker.kt b/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/worker/BackgroundNoiseOneTimeWorker.kt index b77d7c9d79b47591ac2ae4cd42a52eca7ef227fa..522d279d6fd6455a94e7daea9c3d0a8b9d14250f 100644 --- a/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/worker/BackgroundNoiseOneTimeWorker.kt +++ b/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/worker/BackgroundNoiseOneTimeWorker.kt @@ -20,13 +20,8 @@ class BackgroundNoiseOneTimeWorker @AssistedInject constructor( private val playbook: Playbook ) : CoroutineWorker(context, workerParams) { - /** - * Work execution - * - * @return Result - */ override suspend fun doWork(): Result { - Timber.d("$id: doWork() started. Run attempt: $runAttemptCount") + Timber.tag(TAG).d("$id: doWork() started. Run attempt: $runAttemptCount") var result = Result.success() try { @@ -40,10 +35,14 @@ class BackgroundNoiseOneTimeWorker @AssistedInject constructor( } } - Timber.d("$id: doWork() finished with %s", result) + Timber.tag(TAG).d("$id: doWork() finished with %s", result) return result } @AssistedInject.Factory interface Factory : InjectedWorkerFactory<BackgroundNoiseOneTimeWorker> + + companion object { + private val TAG = BackgroundNoiseOneTimeWorker::class.java.simpleName + } } diff --git a/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/worker/BackgroundNoisePeriodicWorker.kt b/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/worker/BackgroundNoisePeriodicWorker.kt index 15b175f853c7dd9ffdc7385c4130b0460d293232..d1111667fe3b73b484fb089198120c81a56593b8 100644 --- a/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/worker/BackgroundNoisePeriodicWorker.kt +++ b/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/worker/BackgroundNoisePeriodicWorker.kt @@ -22,19 +22,11 @@ class BackgroundNoisePeriodicWorker @AssistedInject constructor( @Assisted workerParams: WorkerParameters ) : CoroutineWorker(context, workerParams) { - companion object { - private val TAG: String? = BackgroundNoisePeriodicWorker::class.simpleName - } - /** - * Work execution - * - * @return Result - * * @see BackgroundConstants.NUMBER_OF_DAYS_TO_RUN_PLAYBOOK */ override suspend fun doWork(): Result { - Timber.d("$id: doWork() started. Run attempt: $runAttemptCount") + Timber.tag(TAG).d("$id: doWork() started. Run attempt: $runAttemptCount") var result = Result.success() try { @@ -57,15 +49,19 @@ class BackgroundNoisePeriodicWorker @AssistedInject constructor( Result.retry() } } - Timber.d("$id: doWork() finished with %s", result) + Timber.tag(TAG).d("$id: doWork() finished with %s", result) return result } private fun stopWorker() { BackgroundWorkScheduler.WorkType.BACKGROUND_NOISE_PERIODIC_WORK.stop() - Timber.d("$id: worker stopped") + Timber.tag(TAG).d("$id: worker stopped") } @AssistedInject.Factory interface Factory : InjectedWorkerFactory<BackgroundNoisePeriodicWorker> + + companion object { + private val TAG = BackgroundNoisePeriodicWorker::class.java.simpleName + } } diff --git a/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/worker/DiagnosisKeyRetrievalOneTimeWorker.kt b/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/worker/DiagnosisKeyRetrievalOneTimeWorker.kt index 5546e8947deebec1c3536ed4e206da4e22fbfbc3..23199d02733d645ba7688ce839cb6008126c84c0 100644 --- a/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/worker/DiagnosisKeyRetrievalOneTimeWorker.kt +++ b/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/worker/DiagnosisKeyRetrievalOneTimeWorker.kt @@ -24,13 +24,8 @@ class DiagnosisKeyRetrievalOneTimeWorker @AssistedInject constructor( private val taskController: TaskController ) : CoroutineWorker(context, workerParams) { - /** - * Work execution - * - * @return Result - */ override suspend fun doWork(): Result { - Timber.d("$id: doWork() started. Run attempt: $runAttemptCount") + Timber.tag(TAG).d("$id: doWork() started. Run attempt: $runAttemptCount") var result = Result.success() taskController.submitBlocking( @@ -40,22 +35,26 @@ class DiagnosisKeyRetrievalOneTimeWorker @AssistedInject constructor( originTag = "DiagnosisKeyRetrievalOneTimeWorker" ) ).error?.also { error: Throwable -> - Timber.w(error, "$id: Error when submitting DownloadDiagnosisKeysTask.") + Timber.tag(TAG).w(error, "$id: Error when submitting DownloadDiagnosisKeysTask.") if (runAttemptCount > BackgroundConstants.WORKER_RETRY_COUNT_THRESHOLD) { - Timber.w(error, "$id: Retry attempts exceeded.") + Timber.tag(TAG).w(error, "$id: Retry attempts exceeded.") return Result.failure() } else { - Timber.d(error, "$id: Retrying.") + Timber.tag(TAG).d(error, "$id: Retrying.") result = Result.retry() } } - Timber.d("$id: doWork() finished with %s", result) + Timber.tag(TAG).d("$id: doWork() finished with %s", result) return result } @AssistedInject.Factory interface Factory : InjectedWorkerFactory<DiagnosisKeyRetrievalOneTimeWorker> + + companion object { + private val TAG = DiagnosisKeyRetrievalOneTimeWorker::class.java.simpleName + } } diff --git a/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/worker/DiagnosisKeyRetrievalPeriodicWorker.kt b/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/worker/DiagnosisKeyRetrievalPeriodicWorker.kt index 4bcd5bc8f1a684461d0a1e0d92cf290487c4741e..adff8651b5bb674bad55ecbba08bbccad7b0adab 100644 --- a/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/worker/DiagnosisKeyRetrievalPeriodicWorker.kt +++ b/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/worker/DiagnosisKeyRetrievalPeriodicWorker.kt @@ -21,38 +21,38 @@ class DiagnosisKeyRetrievalPeriodicWorker @AssistedInject constructor( ) : CoroutineWorker(context, workerParams) { /** - * Work execution - * - * @return Result - * * @see BackgroundWorkScheduler.scheduleDiagnosisKeyPeriodicWork() * @see BackgroundWorkScheduler.scheduleDiagnosisKeyOneTimeWork() */ override suspend fun doWork(): Result { - Timber.d("$id: doWork() started. Run attempt: $runAttemptCount") + Timber.tag(TAG).d("$id: doWork() started. Run attempt: $runAttemptCount") var result = Result.success() try { BackgroundWorkScheduler.scheduleDiagnosisKeyOneTimeWork() } catch (e: Exception) { - Timber.w( + Timber.tag(TAG).w( e, "$id: Error during BackgroundWorkScheduler.scheduleDiagnosisKeyOneTimeWork()." ) if (runAttemptCount > BackgroundConstants.WORKER_RETRY_COUNT_THRESHOLD) { - Timber.w(e, "$id: Retry attempts exceeded.") + Timber.tag(TAG).w(e, "$id: Retry attempts exceeded.") return Result.failure() } else { - Timber.d(e, "$id: Retrying.") + Timber.tag(TAG).d(e, "$id: Retrying.") result = Result.retry() } } - Timber.d("$id: doWork() finished with %s", result) + Timber.tag(TAG).d("$id: doWork() finished with %s", result) return result } @AssistedInject.Factory interface Factory : InjectedWorkerFactory<DiagnosisKeyRetrievalPeriodicWorker> + + companion object { + private val TAG = DiagnosisKeyRetrievalPeriodicWorker::class.java.simpleName + } } diff --git a/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/worker/DiagnosisTestResultRetrievalPeriodicWorker.kt b/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/worker/DiagnosisTestResultRetrievalPeriodicWorker.kt index 33eb975e64fa80d20fc2ee29a213be215a2396ba..8b604d2fe06438801f6808bac57d708b5e122612 100644 --- a/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/worker/DiagnosisTestResultRetrievalPeriodicWorker.kt +++ b/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/worker/DiagnosisTestResultRetrievalPeriodicWorker.kt @@ -27,25 +27,20 @@ class DiagnosisTestResultRetrievalPeriodicWorker @AssistedInject constructor( ) : CoroutineWorker(context, workerParams) { /** - * Work execution - * * If background job is running for less than 21 days, testResult is checked. * If the job is running for more than 21 days, the job will be stopped * - * @return Result - * * @see LocalData.isTestResultNotificationSent * @see LocalData.initialPollingForTestResultTimeStamp */ override suspend fun doWork(): Result { - - Timber.d("$id: doWork() started. Run attempt: $runAttemptCount") + Timber.tag(TAG).d("$id: doWork() started. Run attempt: $runAttemptCount") if (runAttemptCount > BackgroundConstants.WORKER_RETRY_COUNT_THRESHOLD) { - Timber.d("$id doWork() failed after $runAttemptCount attempts. Rescheduling") + Timber.tag(TAG).d("$id doWork() failed after $runAttemptCount attempts. Rescheduling") BackgroundWorkScheduler.scheduleDiagnosisKeyPeriodicWork() - Timber.d("$id Rescheduled background worker") + Timber.tag(TAG).d("$id Rescheduled background worker") return Result.failure() } @@ -56,19 +51,19 @@ class DiagnosisTestResultRetrievalPeriodicWorker @AssistedInject constructor( System.currentTimeMillis() ) < BackgroundConstants.POLLING_VALIDITY_MAX_DAYS ) { - Timber.d(" $id maximum days not exceeded") + Timber.tag(TAG).d(" $id maximum days not exceeded") val testResult = SubmissionService.asyncRequestTestResult() initiateNotification(testResult) - Timber.d(" $id Test Result Notification Initiated") + Timber.tag(TAG).d(" $id Test Result Notification Initiated") } else { stopWorker() - Timber.d(" $id worker stopped") + Timber.tag(TAG).d(" $id worker stopped") } } catch (e: Exception) { result = Result.retry() } - Timber.d("$id: doWork() finished with %s", result) + Timber.tag(TAG).d("$id: doWork() finished with %s", result) return result } @@ -86,10 +81,10 @@ class DiagnosisTestResultRetrievalPeriodicWorker @AssistedInject constructor( */ private fun initiateNotification(testResult: TestResult) { if (LocalData.isTestResultNotificationSent() || LocalData.submissionWasSuccessful()) { - Timber.d("$id: Notification already sent or there was a successful submission") + Timber.tag(TAG).d("$id: Notification already sent or there was a successful submission") return } - Timber.d("$id: Test Result retried is $testResult") + Timber.tag(TAG).d("$id: Test Result retried is $testResult") if (testResult == TestResult.NEGATIVE || testResult == TestResult.POSITIVE || testResult == TestResult.INVALID ) { @@ -100,7 +95,7 @@ class DiagnosisTestResultRetrievalPeriodicWorker @AssistedInject constructor( CoronaWarnApplication.getAppContext() .getString(R.string.notification_body) ) - Timber.d("$id: Test Result available and notification is initiated") + Timber.tag(TAG).d("$id: Test Result available and notification is initiated") } LocalData.isTestResultNotificationSent(true) stopWorker() @@ -116,9 +111,13 @@ class DiagnosisTestResultRetrievalPeriodicWorker @AssistedInject constructor( private fun stopWorker() { LocalData.initialPollingForTestResultTimeStamp(0L) BackgroundWorkScheduler.WorkType.DIAGNOSIS_TEST_RESULT_PERIODIC_WORKER.stop() - Timber.d("$id: Background worker stopped") + Timber.tag(TAG).d("$id: Background worker stopped") } @AssistedInject.Factory interface Factory : InjectedWorkerFactory<DiagnosisTestResultRetrievalPeriodicWorker> + + companion object { + private val TAG = DiagnosisTestResultRetrievalPeriodicWorker::class.java.simpleName + } }