[tbb-commits] [Git][tpo/applications/android-components][android-components-96.0.15-11.5-1] 3 commits: Bug 40075: Support scoped storage to enable downloads on API < 29

Matthew Finkel pushed to branch android-components-96.0.15-11.5-1 at The Tor Project / Applications / android-components

Commits: d6444a09 by aguestuser at 2022-02-18T13:22:07-05:00 Bug 40075: Support scoped storage to enable downloads on API &lt; 29

- in android-components!7, we blocked all usage of Scoped Storage in an attempt to block usage of Android&#39;s DownloadManager, which is known to cause proxy bypasses - as of Android API 29, downloads will not work without Scoped Storage, causing all downlaods to fail (see: fenix##40192) - here, we enable usage of scoped storage for API &gt;= 29, but block calls to DownloadManager on API &lt; 29

- - - - - e2daac07 by aguestuser at 2022-02-18T17:09:02-05:00 fixup! Bug 40075: Support scoped storage to enable downloads on API &lt; 29

- abandon attempts to block writing file metadata to system db, as justified in https://gitlab.torproject.org/tpo/applications/android-components/-/issues/4... - this frees us to eliminate the `ANDROID_DOWNLOADS_INTEGRATION` feature toggle altogether

- - - - - 6ae3342f by Matthew Finkel at 2022-02-24T17:26:55+00:00 Merge remote-tracking branch &#39;aguestusergl/bug-40075_support-scoped-storage&#39; into android-components-96.0.15-11.5-1

- - - - -

2 changed files:

- components/feature/downloads/build.gradle - components/feature/downloads/src/main/java/mozilla/components/feature/downloads/AbstractFetchDownloadService.kt

Changes:

===================================== components/feature/downloads/build.gradle ===================================== @@ -21,8 +21,6 @@ android { arg("room.schemaLocation", "$projectDir/schemas".toString()) } } - - buildConfigField("boolean", "ANDROID_DOWNLOADS_INTEGRATION", "false") }

buildTypes {

===================================== components/feature/downloads/src/main/java/mozilla/components/feature/downloads/AbstractFetchDownloadService.kt ===================================== @@ -371,9 +371,7 @@ abstract class AbstractFetchDownloadService : Service() { style.notificationAccentColor ) COMPLETED -> { - if (BuildConfig.ANDROID_DOWNLOADS_INTEGRATION) { - addToDownloadSystemDatabaseCompat(download.state, scope) - } + addToDownloadSystemDatabaseCompat(download.state, scope) DownloadNotification.createDownloadCompletedNotification( context, download, @@ -841,9 +839,7 @@ abstract class AbstractFetchDownloadService : Service() { }

@VisibleForTesting - internal fun shouldUseScopedStorage() = - BuildConfig.ANDROID_DOWNLOADS_INTEGRATION && - getSdkVersion() >= Build.VERSION_CODES.Q + internal fun shouldUseScopedStorage() = getSdkVersion() >= Build.VERSION_CODES.Q

/** * Gets the SDK version from the system.

View it on GitLab: https://gitlab.torproject.org/tpo/applications/android-components/-/compare/...