Repository navigation
ref(android): [minSdk 26 2] Remove code made obsolete by minSdk 26 - #6438
Draft
runningcode wants to merge 3 commits into
Draft
runningcode wants to merge 3 commits into
runningcode wants to merge 3 commits into
Conversation
Remove the API 26 and lower annotations, the runtime checks for API levels below 26 and the branches they guarded, and the bundled ISRG root CA socket factory that only applied to API 25 and lower. Turn the ObsoleteSdkInt lint check back on. Public methods that hybrid SDKs call keep their BuildInfoProvider parameters. Refs JAVA-547 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This was referenced Oct 30, 2026
📲 Install BuildsAndroid
|
This branch has not been deployed
This file contains hidden or 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
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.
PR Stack (minSdk 26)
📜 Description
Removes the code that minSdk 26 (#6435) made dead, and turns the
ObsoleteSdkIntlint check back on.@TargetApi/@RequiresApiannotations for API 26 and lower insentry-android-coreandsentry-android-replayAndroidTransactionProfilerandAndroidContinuousProfilerAndroidConnectionStatusProvider,NetworkBreadcrumbsIntegrationandCellularNetworkTechnologyProvider, including the deprecatedNetworkInfofallbackScreenshotUtils: the canvas fallback for pre-O devices. Screenshots always usePixelCopynow. The public signatures stay as they are because hybrid SDKs call them.SentryFrameMetricsCollector: the API 24 availability check, and the Choreographer frame start fallback for pre-O devicesFirstDrawDoneListener: the pre-26 workaround for the floatingViewTreeObserver. ItsBuildInfoProviderparameter is unused now, so it's removed.AppStartMetrics: the pre-M fallback forMessageQueue.addIdleHandlerReplayIntegration: the "API 26 and above" registration checkSentryRootCaSslSocketFactoryandSentryRootCertificates(fix(android): Trust bundled ISRG Root X1 on API 25 and lower #6227). They only trusted the bundled ISRG Root X1 on API 25 and lower.@SuppressLint("NewApi")suppressions stay. Lint does not flag them, and many cover APIs above 26.💡 Motivation and Context
Follow-up to #6435. This touches files that change often on
main, so it should land close to the v9 release to keep merges frommaininto9.x.xsimple.💚 How did you test it?
lintforsentry-android-core,sentry-android-replayandsentry-android-ndkwithObsoleteSdkIntback onsentry-android-coreandsentry-android-replay📝 Checklist
sendDefaultPIIis enabled.🔮 Next steps
#skip-changelog