firebase-android-sdk icon indicating copy to clipboard operation
firebase-android-sdk copied to clipboard

Resolve StrictMode violation in App Check.

Open rosalyntan opened this issue 3 years ago • 5 comments

Resolve a StrictMode violation in App Check by moving the retrieval of the stored App Check token out of the initialization path.

rosalyntan avatar Sep 13 '22 23:09 rosalyntan

Size Report 1

Affected Products

  • firebase-appcheck

    TypeBase (6df92b9)Merge (5275ef4)Diff
    aar37.1 kB37.3 kB+202 B (+0.5%)
    apk (aggressive)332 kB332 kB+136 B (+0.0%)
    apk (release)977 kB977 kB+768 B (+0.1%)

Test Logs

google-oss-bot avatar Sep 14 '22 00:09 google-oss-bot

Coverage Report 1

Affected Products

  • firebase-appcheck

    Overall coverage changed from 81.39% (6df92b9) to 81.84% (5275ef4) by +0.45%.

    FilenameBase (6df92b9)Merge (5275ef4)Diff
    DefaultFirebaseAppCheck.java81.63%83.62%+1.99%

Test Logs

google-oss-bot avatar Sep 14 '22 00:09 google-oss-bot

Unit Test Results

232 tests  +232   232 :heavy_check_mark: +232   1m 28s :stopwatch: + 1m 28s   40 suites +  40       0 :zzz: ±    0    40 files   +  40       0 :x: ±    0 

Results for commit ef3dbb15. ± Comparison against base commit e6226b74.

:recycle: This comment has been updated with latest results.

github-actions[bot] avatar Sep 14 '22 00:09 github-actions[bot]

@vkryachko are the Firebase Storage unit tests known to be flaky? They all pass every time when I run them locally, but they seem to be failing in the presubmits here (specifically UploadTest). I got the unit test presubmit to run on previous commits after re-running them several times, but the Health Metrics / Coverage tests are consistently failing.

rosalyntan avatar Sep 16 '22 23:09 rosalyntan

Yes, storage has been really flaky lately, Health Metrics are non-blocking, so no need to worry about them

vkryachko avatar Sep 19 '22 17:09 vkryachko