-
-
Notifications
You must be signed in to change notification settings - Fork 338
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
chore(deps): update Android SDK to v7.3.0 #3434
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
github-actions
bot
requested review from
krystofwoldrich and
lucas-zimerman
as code owners
November 30, 2023 13:50
github-actions
bot
added
the
dependencies
Pull requests that update a dependency file
label
Nov 30, 2023
bruno-garcia
force-pushed
the
deps/scripts/update-android.sh
branch
from
November 30, 2023 13:50
b5bd422
to
265982c
Compare
iOS (legacy) Performance metrics 🚀
|
Revision | Plain | With Sentry | Diff |
---|---|---|---|
9433f35+dirty | 1246.94 ms | 1271.45 ms | 24.52 ms |
9c48b2c+dirty | 1246.96 ms | 1255.73 ms | 8.77 ms |
1d86dd6+dirty | 1249.71 ms | 1279.16 ms | 29.45 ms |
d361d38+dirty | 1246.04 ms | 1267.12 ms | 21.08 ms |
12427f4+dirty | 1267.15 ms | 1271.30 ms | 4.15 ms |
6e8584e+dirty | 1274.50 ms | 1296.82 ms | 22.32 ms |
3ffcddd+dirty | 1244.47 ms | 1264.14 ms | 19.67 ms |
22e31b6+dirty | 1253.62 ms | 1265.96 ms | 12.34 ms |
0677344+dirty | 1276.70 ms | 1300.07 ms | 23.37 ms |
3853f43+dirty | 1221.82 ms | 1242.64 ms | 20.82 ms |
App size
Revision | Plain | With Sentry | Diff |
---|---|---|---|
9433f35+dirty | 2.36 MiB | 2.85 MiB | 499.80 KiB |
9c48b2c+dirty | 2.36 MiB | 2.85 MiB | 495.77 KiB |
1d86dd6+dirty | 2.36 MiB | 2.89 MiB | 535.43 KiB |
d361d38+dirty | 2.36 MiB | 2.85 MiB | 499.84 KiB |
12427f4+dirty | 2.36 MiB | 2.88 MiB | 530.38 KiB |
6e8584e+dirty | 2.36 MiB | 2.88 MiB | 533.17 KiB |
3ffcddd+dirty | 2.36 MiB | 2.84 MiB | 489.60 KiB |
22e31b6+dirty | 2.36 MiB | 2.87 MiB | 520.67 KiB |
0677344+dirty | 2.36 MiB | 2.85 MiB | 496.81 KiB |
3853f43+dirty | 2.36 MiB | 2.85 MiB | 499.81 KiB |
Previous results on branch: deps/scripts/update-android.sh
Startup times
Revision | Plain | With Sentry | Diff |
---|---|---|---|
111c58e+dirty | 1265.76 ms | 1271.32 ms | 5.56 ms |
6222f43+dirty | 1247.86 ms | 1262.94 ms | 15.08 ms |
80e039e+dirty | 1218.47 ms | 1228.74 ms | 10.27 ms |
8d763c2+dirty | 1256.46 ms | 1284.62 ms | 28.16 ms |
107b129+dirty | 1257.08 ms | 1267.98 ms | 10.90 ms |
59de53f+dirty | 1254.08 ms | 1276.02 ms | 21.94 ms |
2e9a42f+dirty | 1274.18 ms | 1284.52 ms | 10.34 ms |
fa42167+dirty | 1239.02 ms | 1239.22 ms | 0.20 ms |
ad7319a+dirty | 1215.82 ms | 1236.38 ms | 20.56 ms |
cd98a44+dirty | 1253.20 ms | 1273.48 ms | 20.28 ms |
App size
Revision | Plain | With Sentry | Diff |
---|---|---|---|
111c58e+dirty | 2.36 MiB | 2.85 MiB | 496.85 KiB |
6222f43+dirty | 2.36 MiB | 2.88 MiB | 532.92 KiB |
80e039e+dirty | 2.36 MiB | 2.88 MiB | 531.89 KiB |
8d763c2+dirty | 2.36 MiB | 2.88 MiB | 533.06 KiB |
107b129+dirty | 2.36 MiB | 2.82 MiB | 469.43 KiB |
59de53f+dirty | 2.36 MiB | 2.87 MiB | 520.66 KiB |
2e9a42f+dirty | 2.36 MiB | 2.85 MiB | 499.83 KiB |
fa42167+dirty | 2.36 MiB | 2.85 MiB | 495.31 KiB |
ad7319a+dirty | 2.36 MiB | 2.87 MiB | 520.67 KiB |
cd98a44+dirty | 2.36 MiB | 2.87 MiB | 520.20 KiB |
iOS (new) Performance metrics 🚀
|
Revision | Plain | With Sentry | Diff |
---|---|---|---|
9433f35+dirty | 1232.24 ms | 1232.74 ms | 0.50 ms |
9c48b2c+dirty | 1253.39 ms | 1256.30 ms | 2.91 ms |
1d86dd6+dirty | 1289.25 ms | 1293.36 ms | 4.11 ms |
d361d38+dirty | 1272.96 ms | 1291.70 ms | 18.74 ms |
12427f4+dirty | 1224.90 ms | 1231.40 ms | 6.50 ms |
6e8584e+dirty | 1271.71 ms | 1281.26 ms | 9.55 ms |
3ffcddd+dirty | 1272.22 ms | 1273.98 ms | 1.76 ms |
22e31b6+dirty | 1276.55 ms | 1278.12 ms | 1.57 ms |
0677344+dirty | 1252.52 ms | 1254.08 ms | 1.56 ms |
3853f43+dirty | 1271.74 ms | 1278.04 ms | 6.30 ms |
App size
Revision | Plain | With Sentry | Diff |
---|---|---|---|
9433f35+dirty | 2.92 MiB | 3.41 MiB | 503.55 KiB |
9c48b2c+dirty | 2.92 MiB | 3.41 MiB | 499.97 KiB |
1d86dd6+dirty | 2.92 MiB | 3.44 MiB | 538.27 KiB |
d361d38+dirty | 2.92 MiB | 3.41 MiB | 503.57 KiB |
12427f4+dirty | 2.92 MiB | 3.44 MiB | 533.29 KiB |
6e8584e+dirty | 2.92 MiB | 3.44 MiB | 536.52 KiB |
3ffcddd+dirty | 2.92 MiB | 3.40 MiB | 494.39 KiB |
22e31b6+dirty | 2.92 MiB | 3.43 MiB | 524.74 KiB |
0677344+dirty | 2.92 MiB | 3.41 MiB | 500.94 KiB |
3853f43+dirty | 2.92 MiB | 3.41 MiB | 503.54 KiB |
Previous results on branch: deps/scripts/update-android.sh
Startup times
Revision | Plain | With Sentry | Diff |
---|---|---|---|
111c58e+dirty | 1219.08 ms | 1238.76 ms | 19.67 ms |
6222f43+dirty | 1259.68 ms | 1261.64 ms | 1.96 ms |
80e039e+dirty | 1266.88 ms | 1275.52 ms | 8.64 ms |
8d763c2+dirty | 1296.18 ms | 1311.02 ms | 14.84 ms |
107b129+dirty | 1261.74 ms | 1269.16 ms | 7.42 ms |
59de53f+dirty | 1288.46 ms | 1293.40 ms | 4.95 ms |
2e9a42f+dirty | 1248.31 ms | 1248.52 ms | 0.21 ms |
fa42167+dirty | 1290.73 ms | 1315.08 ms | 24.35 ms |
ad7319a+dirty | 1235.78 ms | 1239.62 ms | 3.84 ms |
cd98a44+dirty | 1303.06 ms | 1307.82 ms | 4.76 ms |
App size
Revision | Plain | With Sentry | Diff |
---|---|---|---|
111c58e+dirty | 2.92 MiB | 3.41 MiB | 501.00 KiB |
6222f43+dirty | 2.92 MiB | 3.44 MiB | 536.26 KiB |
80e039e+dirty | 2.92 MiB | 3.44 MiB | 535.20 KiB |
8d763c2+dirty | 2.92 MiB | 3.44 MiB | 536.46 KiB |
107b129+dirty | 2.92 MiB | 3.38 MiB | 475.70 KiB |
59de53f+dirty | 2.92 MiB | 3.43 MiB | 524.73 KiB |
2e9a42f+dirty | 2.92 MiB | 3.41 MiB | 503.53 KiB |
fa42167+dirty | 2.92 MiB | 3.41 MiB | 499.08 KiB |
ad7319a+dirty | 2.92 MiB | 3.43 MiB | 524.67 KiB |
cd98a44+dirty | 2.92 MiB | 3.43 MiB | 524.25 KiB |
bruno-garcia
force-pushed
the
deps/scripts/update-android.sh
branch
8 times, most recently
from
December 7, 2023 10:51
fa38d56
to
9bce062
Compare
github-actions
bot
changed the title
chore(deps): update Android SDK to v7.0.0
chore(deps): update Android SDK to v7.1.0
Dec 19, 2023
bruno-garcia
force-pushed
the
deps/scripts/update-android.sh
branch
3 times, most recently
from
December 22, 2023 13:09
13efed6
to
6147861
Compare
bruno-garcia
force-pushed
the
deps/scripts/update-android.sh
branch
6 times, most recently
from
January 10, 2024 14:25
6871573
to
fc48aec
Compare
bruno-garcia
force-pushed
the
deps/scripts/update-android.sh
branch
from
January 12, 2024 14:06
fc48aec
to
a0f799d
Compare
github-actions
bot
changed the title
chore(deps): update Android SDK to v7.1.0
chore(deps): update Android SDK to v7.2.0
Jan 12, 2024
bruno-garcia
force-pushed
the
deps/scripts/update-android.sh
branch
4 times, most recently
from
January 18, 2024 09:43
8215d44
to
99b71b4
Compare
bruno-garcia
force-pushed
the
deps/scripts/update-android.sh
branch
10 times, most recently
from
January 25, 2024 05:09
1d72257
to
58cc017
Compare
bruno-garcia
force-pushed
the
deps/scripts/update-android.sh
branch
2 times, most recently
from
January 30, 2024 18:21
1c60eee
to
b6b131d
Compare
github-actions
bot
changed the title
chore(deps): update Android SDK to v7.2.0
chore(deps): update Android SDK to v7.3.0
Jan 30, 2024
bruno-garcia
force-pushed
the
deps/scripts/update-android.sh
branch
5 times, most recently
from
February 1, 2024 15:26
636533b
to
001a071
Compare
bruno-garcia
force-pushed
the
deps/scripts/update-android.sh
branch
from
February 5, 2024 08:39
001a071
to
0fadf67
Compare
Android (legacy) Performance metrics 🚀
|
Revision | Plain | With Sentry | Diff |
---|---|---|---|
728164b | 414.34 ms | 449.22 ms | 34.88 ms |
34aba08 | 328.10 ms | 342.84 ms | 14.74 ms |
b1e8712 | 462.11 ms | 465.71 ms | 3.60 ms |
e73f4ed+dirty | 332.96 ms | 354.33 ms | 21.37 ms |
9433f35 | 347.64 ms | 356.22 ms | 8.58 ms |
457e29f | 398.10 ms | 421.39 ms | 23.29 ms |
86d6d2c+dirty | 332.90 ms | 352.45 ms | 19.55 ms |
3853f43 | 329.68 ms | 346.32 ms | 16.64 ms |
2534337 | 394.15 ms | 415.12 ms | 20.97 ms |
3ffcddd | 302.92 ms | 315.80 ms | 12.88 ms |
App size
Revision | Plain | With Sentry | Diff |
---|---|---|---|
728164b | 17.73 MiB | 19.85 MiB | 2.12 MiB |
34aba08 | 17.73 MiB | 19.80 MiB | 2.07 MiB |
b1e8712 | 17.73 MiB | 19.75 MiB | 2.02 MiB |
e73f4ed+dirty | 17.73 MiB | 20.04 MiB | 2.31 MiB |
9433f35 | 17.73 MiB | 19.81 MiB | 2.08 MiB |
457e29f | 17.73 MiB | 19.84 MiB | 2.10 MiB |
86d6d2c+dirty | 17.73 MiB | 20.04 MiB | 2.31 MiB |
3853f43 | 17.73 MiB | 19.81 MiB | 2.08 MiB |
2534337 | 17.73 MiB | 19.84 MiB | 2.11 MiB |
3ffcddd | 17.73 MiB | 19.75 MiB | 2.02 MiB |
Previous results on branch: deps/scripts/update-android.sh
Startup times
Revision | Plain | With Sentry | Diff |
---|---|---|---|
bf931f5+dirty | 346.46 ms | 394.20 ms | 47.75 ms |
fa42167 | 335.87 ms | 355.96 ms | 20.09 ms |
e570815 | 410.80 ms | 430.00 ms | 19.20 ms |
6222f43 | 420.96 ms | 446.28 ms | 25.32 ms |
59de53f | 388.17 ms | 435.58 ms | 47.41 ms |
2e9a42f | 358.96 ms | 384.77 ms | 25.81 ms |
111c58e | 313.04 ms | 316.54 ms | 3.50 ms |
80e039e | 403.14 ms | 426.94 ms | 23.80 ms |
cd98a44 | 491.98 ms | 596.15 ms | 104.17 ms |
ae87f29 | 442.48 ms | 466.00 ms | 23.52 ms |
App size
Revision | Plain | With Sentry | Diff |
---|---|---|---|
bf931f5+dirty | 17.73 MiB | 19.75 MiB | 2.02 MiB |
fa42167 | 17.73 MiB | 19.80 MiB | 2.07 MiB |
e570815 | 17.73 MiB | 19.85 MiB | 2.12 MiB |
6222f43 | 17.73 MiB | 19.86 MiB | 2.12 MiB |
59de53f | 17.73 MiB | 19.84 MiB | 2.10 MiB |
2e9a42f | 17.73 MiB | 19.81 MiB | 2.08 MiB |
111c58e | 17.73 MiB | 19.81 MiB | 2.07 MiB |
80e039e | 17.73 MiB | 19.85 MiB | 2.12 MiB |
cd98a44 | 17.73 MiB | 19.83 MiB | 2.10 MiB |
ae87f29 | 17.73 MiB | 19.85 MiB | 2.12 MiB |
Android (new) Performance metrics 🚀
|
Revision | Plain | With Sentry | Diff |
---|---|---|---|
5446992+dirty | 371.61 ms | 390.00 ms | 18.39 ms |
728164b+dirty | 335.93 ms | 342.94 ms | 7.01 ms |
1d86dd6+dirty | 335.76 ms | 371.22 ms | 35.46 ms |
e73f4ed+dirty | 262.98 ms | 311.02 ms | 48.04 ms |
ad6c299+dirty | 336.47 ms | 362.89 ms | 26.42 ms |
9433f35+dirty | 265.50 ms | 336.08 ms | 70.58 ms |
457e29f+dirty | 591.49 ms | 612.96 ms | 21.47 ms |
86d6d2c+dirty | 267.21 ms | 325.24 ms | 58.04 ms |
12427f4+dirty | 379.48 ms | 400.92 ms | 21.44 ms |
abb7058+dirty | 320.78 ms | 324.08 ms | 3.30 ms |
App size
Revision | Plain | With Sentry | Diff |
---|---|---|---|
5446992+dirty | 7.15 MiB | 8.12 MiB | 999.45 KiB |
728164b+dirty | 7.15 MiB | 8.12 MiB | 997.71 KiB |
1d86dd6+dirty | 7.15 MiB | 8.13 MiB | 1002.18 KiB |
e73f4ed+dirty | 7.15 MiB | 8.09 MiB | 965.94 KiB |
ad6c299+dirty | 7.15 MiB | 8.04 MiB | 912.17 KiB |
9433f35+dirty | 7.15 MiB | 8.08 MiB | 959.34 KiB |
457e29f+dirty | 7.15 MiB | 8.10 MiB | 981.29 KiB |
86d6d2c+dirty | 7.15 MiB | 8.09 MiB | 962.69 KiB |
12427f4+dirty | 7.15 MiB | 8.12 MiB | 997.78 KiB |
abb7058+dirty | 7.15 MiB | 8.10 MiB | 980.40 KiB |
Previous results on branch: deps/scripts/update-android.sh
Startup times
Revision | Plain | With Sentry | Diff |
---|---|---|---|
bf931f5+dirty | 257.49 ms | 300.92 ms | 43.43 ms |
8d763c2+dirty | 352.61 ms | 377.30 ms | 24.69 ms |
2e9a42f+dirty | 281.55 ms | 334.18 ms | 52.63 ms |
6222f43+dirty | 339.49 ms | 379.51 ms | 40.02 ms |
80e039e+dirty | 334.59 ms | 352.43 ms | 17.84 ms |
cd98a44+dirty | 345.19 ms | 374.66 ms | 29.47 ms |
fa42167+dirty | 270.35 ms | 291.58 ms | 21.23 ms |
111c58e+dirty | 265.72 ms | 332.72 ms | 67.00 ms |
59de53f+dirty | 362.75 ms | 400.38 ms | 37.63 ms |
ae87f29+dirty | 369.92 ms | 407.58 ms | 37.66 ms |
App size
Revision | Plain | With Sentry | Diff |
---|---|---|---|
bf931f5+dirty | 7.15 MiB | 8.04 MiB | 910.84 KiB |
8d763c2+dirty | 7.15 MiB | 8.13 MiB | 1002.69 KiB |
2e9a42f+dirty | 7.15 MiB | 8.08 MiB | 959.34 KiB |
6222f43+dirty | 7.15 MiB | 8.13 MiB | 1002.31 KiB |
80e039e+dirty | 7.15 MiB | 8.12 MiB | 999.91 KiB |
cd98a44+dirty | 7.15 MiB | 8.10 MiB | 979.68 KiB |
fa42167+dirty | 7.15 MiB | 8.07 MiB | 947.70 KiB |
111c58e+dirty | 7.15 MiB | 8.07 MiB | 950.00 KiB |
59de53f+dirty | 7.15 MiB | 8.11 MiB | 983.75 KiB |
ae87f29+dirty | 7.15 MiB | 8.12 MiB | 999.34 KiB |
krystofwoldrich
approved these changes
Feb 5, 2024
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.
LGTM 🚀
We can merge after CI is done.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Bumps scripts/update-android.sh from 6.34.0 to 7.3.0.
Auto-generated by a dependency updater.
Changelog
7.3.0
Features
io.sentry.profiling.enable-app-start
, other than the already existingio.sentry.traces.profiling.sample-rate
.isForNextAppStart
flag, to adjust startup profiling sampling programmatically.Relevant PRs:
OnFrameMetricsAvailableListener
(#3111)Fixes
mechanism:AppExitInfo
Breaking changes
HostnameVerifier
option as it's flagged by security tools of some app stores (#3150)ITransportFactory
throughSentryOptions.setTransportFactory()
, where you can copy over most of the parts likeHttpConnection
andAsyncHttpTransport
from the SDK with necessary modificationsDependencies
7.2.0
Features
monitor
/check_in
in client reports and rate limiter (#3096)graphql-java
version 21 (#3090)Fixes
SentryJobListener
release
and other fields for ANRs reported withmechanism:AppExitInfo
(#3074)Dependencies
opentelemetry-sdk
to1.33.0
andopentelemetry-javaagent
to1.32.0
(#3112)7.1.0
Features
enable-backpressure-handling=true
.tracesSampleRate
automatically.tracesSampleRate
to its original value.4.1.0
or neweroptions.isEnablePerformanceV2 = true
Fixes
SentryOkHttpEventListener
even without transactions (#3087)io.sentry.exception.SentryHttpClientException
from obfuscation to display proper issue title on Sentry (#3093)Dependencies
7.0.0
Version 7 of the Sentry Android/Java SDK brings a variety of features and fixes. The most notable changes are:
minSdk
level to 19 (Android 4.4)Sentry.getSpan
now returns the root transaction, which should improve the span hierarchy and make it leanersentry-okhttp
artifact is unbundled from Android and can be used in pure JVM-only appsSentry Self-hosted Compatibility
This SDK version is compatible with a self-hosted version of Sentry
22.12.0
or higher. If you are using an older version of self-hosted Sentry (aka onpremise), you will need to upgrade. If you're usingsentry.io
no action is required.Sentry Integrations Version Compatibility (Android)
Make sure to align all Sentry dependencies to the same version when bumping the SDK to 7.+, otherwise it will crash at runtime due to binary incompatibility. (E.g. if you're using
-timber
,-okhttp
or other packages)For example, if you're using the Sentry Android Gradle plugin with the
autoInstallation
feature (enabled by default), make sure to use version 4.+ of the gradle plugin together with version 7.+ of the SDK. If you can't do that for some reason, you can specify sentry version via the plugin config block:sentry { autoInstallation { sentryVersion.set("7.0.0") } }
Similarly, if you have a Sentry SDK (e.g.
sentry-android-core
) dependency on one of your Gradle modules and you're updating it to 7.+, make sure the Gradle plugin is at 4.+ or specify the SDK version as shown in the snippet above.Breaking Changes
sentry-kotlin-extensions
, it requireskotlinx-coroutines-core
version1.6.1
or higher now (#2838)SentryOkHttpUtils
was removed from public API as it's been exposed by mistake (#3005)Scope
now implements theIScope
interface, therefore some methods likeScopeCallback.run
acceptIScope
now (#3066)startTransaction
overloads (#2964)TransactionOptions
object instead of having separate parameters for certain optionsTransactionOptions
has defaults set and can be customized, for example:Behavioural Changes
Sentry.getSpan()
returns the root span/transaction instead of the latest span (#2855)sentry-android-okhttp
orsentry-apollo-3
integrations. You can customize what errors you want/don't want to have reported for OkHttp and Apollo3 respectively.onResume
(#2851)deadline_exceeded
when reaching the deadlinesendDefaultPii
if anencoder
has been configured (#2976)maxSpans
setting (defaults to 1000) is enforced for nested child spans which means a single transaction can havemaxSpans
number of children (nested or not) at most (#3065)ScopeCallback
inwithScope
is now always executed (#3066)Deprecations
sentry-android-okhttp
was deprecated in favour of the newsentry-okhttp
module. Make sure to replaceio.sentry.android.okhttp
package name withio.sentry.okhttp
before the next major, where the classes will be removed (#3005)Other Changes
Features
SendCachedEnvelopeIntegration
SendCachedEnvelopeFireAndForgetIntegration
as mentioned in https://docs.sentry.io/platforms/java/configuration/#configuring-offline-caching and provide your own implementation ofIConnectionStatusProvider
viaSentryOptions
sentry-okhttp
module to support instrumenting OkHttp in non-Android projects (#3005)Fixes
getMyMemoryState()
instead ofgetRunningAppProcesses()
to retrieve process importance (#3004)aborted
in case a crash happens (#2938)default
interface methods to proper implementations in each interface implementor (#2847)SentryOkHttpInterceptor.BeforeSpanCallback
was not finishing span when it was dropped (#2958)