Merge branch 'gabr/fixsha' into 'master'
DownloadWorker, really fail on incorrect hashes See merge request AuroraOSS/AuroraStore!354
This commit is contained in:
@@ -298,8 +298,13 @@ class DownloadWorker @AssistedInject constructor(
|
|||||||
}
|
}
|
||||||
|
|
||||||
// Ensure downloaded file matches expected sha
|
// Ensure downloaded file matches expected sha
|
||||||
assert(validSha(request.file, expectedSha, algorithm))
|
if (validSha(request.file, expectedSha, algorithm)) {
|
||||||
return@withContext Result.success()
|
return@withContext Result.success()
|
||||||
|
} else {
|
||||||
|
Log.e(TAG, "Incorrect hash for ${request.file}")
|
||||||
|
throw Exception("Incorrect hash")
|
||||||
|
}
|
||||||
|
|
||||||
} catch (exception: Exception) {
|
} catch (exception: Exception) {
|
||||||
Log.e(TAG, "Failed to download ${request.file}!", exception)
|
Log.e(TAG, "Failed to download ${request.file}!", exception)
|
||||||
notifyStatus(DownloadStatus.FAILED)
|
notifyStatus(DownloadStatus.FAILED)
|
||||||
|
|||||||
Reference in New Issue
Block a user