-
-
Notifications
You must be signed in to change notification settings - Fork 233
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
block app start if appLaunchedInForeground is not set. #2291
base: main
Are you sure you want to change the base?
block app start if appLaunchedInForeground is not set. #2291
Conversation
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2291 +/- ##
==========================================
+ Coverage 88.03% 90.71% +2.68%
==========================================
Files 247 73 -174
Lines 8585 2370 -6215
==========================================
- Hits 7558 2150 -5408
+ Misses 1027 220 -807 ☔ View full report in Codecov by Sentry. |
iOS Performance metrics 🚀
|
Revision | Plain | With Sentry | Diff |
---|---|---|---|
c3e6c82 | 1256.93 ms | 1276.17 ms | 19.24 ms |
a7acb24 | 1296.71 ms | 1317.69 ms | 20.98 ms |
280ab9f | 1255.71 ms | 1262.78 ms | 7.06 ms |
0be962b | 1264.10 ms | 1281.16 ms | 17.06 ms |
7f75f32 | 1230.80 ms | 1251.79 ms | 20.99 ms |
014c3ea | 1298.73 ms | 1351.24 ms | 52.51 ms |
9811573 | 1259.78 ms | 1278.33 ms | 18.55 ms |
b47809a | 1252.61 ms | 1278.57 ms | 25.96 ms |
f172c4d | 1350.66 ms | 1408.49 ms | 57.83 ms |
6f3717a | 1259.84 ms | 1280.90 ms | 21.06 ms |
App size
Revision | Plain | With Sentry | Diff |
---|---|---|---|
c3e6c82 | 8.32 MiB | 9.38 MiB | 1.06 MiB |
a7acb24 | 8.16 MiB | 9.17 MiB | 1.01 MiB |
280ab9f | 8.29 MiB | 9.39 MiB | 1.10 MiB |
0be962b | 8.10 MiB | 9.16 MiB | 1.07 MiB |
7f75f32 | 8.28 MiB | 9.33 MiB | 1.05 MiB |
014c3ea | 8.33 MiB | 9.39 MiB | 1.06 MiB |
9811573 | 8.16 MiB | 9.17 MiB | 1.01 MiB |
b47809a | 8.16 MiB | 9.17 MiB | 1.01 MiB |
f172c4d | 8.33 MiB | 9.62 MiB | 1.29 MiB |
6f3717a | 8.33 MiB | 9.62 MiB | 1.29 MiB |
Previous results on branch: feat/use-app-launched-in-foreground-to-determinate-invalid-app-start-data-on-android
Startup times
Revision | Plain | With Sentry | Diff |
---|---|---|---|
cf64268 | 1249.25 ms | 1274.68 ms | 25.43 ms |
b2dc712 | 1240.17 ms | 1260.31 ms | 20.14 ms |
App size
Revision | Plain | With Sentry | Diff |
---|---|---|---|
cf64268 | 8.38 MiB | 9.74 MiB | 1.36 MiB |
b2dc712 | 8.38 MiB | 9.74 MiB | 1.36 MiB |
Android Performance metrics 🚀
|
Revision | Plain | With Sentry | Diff |
---|---|---|---|
c328ffc | 394.35 ms | 480.94 ms | 86.59 ms |
5e8d2b3 | 342.17 ms | 375.83 ms | 33.66 ms |
8d64376 | 302.88 ms | 356.84 ms | 53.96 ms |
4477d2e | 392.75 ms | 472.69 ms | 79.94 ms |
9811573 | 309.76 ms | 402.60 ms | 92.84 ms |
31b2afb | 397.04 ms | 475.09 ms | 78.04 ms |
d0312c9 | 354.78 ms | 411.53 ms | 56.75 ms |
1c6eb5b | 350.69 ms | 393.86 ms | 43.17 ms |
2331d89 | 352.45 ms | 417.34 ms | 64.89 ms |
7f2b01d | 304.94 ms | 345.71 ms | 40.78 ms |
App size
Revision | Plain | With Sentry | Diff |
---|---|---|---|
c328ffc | 6.35 MiB | 7.42 MiB | 1.07 MiB |
5e8d2b3 | 6.15 MiB | 7.13 MiB | 1000.11 KiB |
8d64376 | 5.94 MiB | 6.96 MiB | 1.02 MiB |
4477d2e | 6.33 MiB | 7.26 MiB | 950.38 KiB |
9811573 | 5.94 MiB | 6.96 MiB | 1.02 MiB |
31b2afb | 6.34 MiB | 7.28 MiB | 966.36 KiB |
d0312c9 | 6.33 MiB | 7.26 MiB | 949.76 KiB |
1c6eb5b | 5.94 MiB | 6.92 MiB | 1001.53 KiB |
2331d89 | 5.94 MiB | 6.96 MiB | 1.02 MiB |
7f2b01d | 5.94 MiB | 6.95 MiB | 1.01 MiB |
Previous results on branch: feat/use-app-launched-in-foreground-to-determinate-invalid-app-start-data-on-android
Startup times
Revision | Plain | With Sentry | Diff |
---|---|---|---|
b2dc712 | 455.00 ms | 516.40 ms | 61.40 ms |
cf64268 | 540.74 ms | 638.23 ms | 97.49 ms |
App size
Revision | Plain | With Sentry | Diff |
---|---|---|---|
b2dc712 | 6.49 MiB | 7.55 MiB | 1.07 MiB |
cf64268 | 6.49 MiB | 7.55 MiB | 1.07 MiB |
…minate-invalid-app-start-data-on-android
…lid-app-start-data-on-android' of https://github.com/getsentry/sentry-dart into feat/use-app-launched-in-foreground-to-determinate-invalid-app-start-data-on-android * 'feat/use-app-launched-in-foreground-to-determinate-invalid-app-start-data-on-android' of https://github.com/getsentry/sentry-dart: test: rethrow exceptions (#2290)
…minate-invalid-app-start-data-on-android
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good
@stefanosiano can you double check if this is the correct behaviour we want?
flutter/android/src/main/kotlin/io/sentry/flutter/SentryFlutterPlugin.kt
Outdated
Show resolved
Hide resolved
…rPlugin.kt Co-authored-by: Giancarlo Buenaflor <[email protected]>
…minate-invalid-app-start-data-on-android
📜 Description
Blocking app starts if "appLaunchedInForeground" is false.
💡 Motivation and Context
close #2215
💚 How did you test it?
📝 Checklist
sendDefaultPii
is enabled🔮 Next steps