fix: follow android 11 storage permissions (#66)

.github/workflows/build.yml

@@ -268,6 +268,7 @@ jobs:
           gomobile init
           gomobile bind -tags nosqlite -ldflags="-w -s" -o ui/flutter/android/app/libs/libgopeed.aar -target=android -androidapi 19 -javapkg=com.gopeed github.com/monkeyWie/gopeed/bind/mobile
           cd ui/flutter
+          echo $APK_KEYSTORE | base64 -d > android/app/upload-keystore.jks
           flutter build apk
           mkdir dist
           cp build/app/outputs/flutter-apk/app-release.apk dist/Gopeed-$VERSION-android.apk
@@ -324,13 +325,25 @@ jobs:
     needs: [get-release]
     steps:
       - uses: actions/checkout@v3
-      - name: Build
+      - name: Set up QEMU
+        uses: docker/setup-qemu-action@v2
+      - name: Set up Docker Buildx
+        uses: docker/setup-buildx-action@v2
+      - name: Login to DockerHub
+        uses: docker/login-action@v1
+        with:
+          username: liwei2633
+          password: ${{ secrets.DOCKER_PASSWORD }}
+      - name: Build and push
+        uses: docker/build-push-action@v2
         env:
           VERSION: ${{ needs.get-release.outputs.tag_name }}
-        run: |
+        with:
-          tag=liwei2633/gopeed:${VERSION:1}
+          push: true
-          docker build -t $tag .
+          platforms: |
-          docker login -u liwei2633 -p ${{ secrets.DOCKER_PASSWORD }}
+            linux/386
-          docker push $tag
+            linux/amd64
-          docker tag $tag liwei2633/gopeed:latest
+            linux/arm64
-          docker push liwei2633/gopeed:latest
+          tags: |
+            liwei2633/gopeed:latest
+            liwei2633/gopeed:${{ env.VERSION }}

pkg/download/downloader.go

@@ -102,14 +102,9 @@ func (d *Downloader) Setup() error {
 
 						current := task.fetcher.Progress().TotalDownloaded()
 						task.Progress.Used = task.timer.Used()
-						downloaded := current - task.Progress.Downloaded
+						currUsedTime := float64(d.refreshInterval) / 1000
-						usedTime := float64(d.refreshInterval) / 1000
+						currSpeed := int64(float64(current-task.Progress.Downloaded) / currUsedTime)
-						if downloaded == 0 {
+						task.Progress.Speed = (task.Progress.Speed + currSpeed) / 2
-							task.Progress.Speed = int64(float64(task.Progress.Speed) / (usedTime * 1.55))
-						} else {
-							usedTime := float64(d.refreshInterval) / 1000
-							task.Progress.Speed = int64(float64(current-task.Progress.Downloaded) / usedTime)
-						}
 						task.Progress.Downloaded = current
 						d.emit(EventKeyProgress, task)
 

pkg/download/model.go

@@ -34,7 +34,7 @@ func (cfg *DownloaderConfig) Init() *DownloaderConfig {
 	}
 
 	if cfg.RefreshInterval == 0 {
-		cfg.RefreshInterval = 500
+		cfg.RefreshInterval = 350
 	}
 	return cfg
 }

ui/flutter/android/app/build.gradle

@@ -53,11 +53,22 @@ android {
         versionName flutterVersionName
     }
 
+    signingConfigs {
+        release {
+            keyAlias "upload"
+            keyPassword System.getenv("APK_KEY_PASSWORD")
+            storeFile file("upload-keystore.jks")
+            storePassword System.getenv("APK_STORE_PASSWORD")
+        }
+    }
+
     buildTypes {
         release {
-            // TODO: Add your own signing config for the release build.
+            if (signingConfigs.release.keyPassword != null) {
-            // Signing with the debug keys for now, so `flutter run --release` works.
+                signingConfig signingConfigs.release
-            signingConfig signingConfigs.debug
+            } else {
+                signingConfig signingConfigs.debug
+            }
         }
     }
 

ui/flutter/android/app/src/debug/AndroidManifest.xml

@@ -4,9 +4,5 @@
          the Flutter tool needs it to communicate with the running application
          to allow setting breakpoints, to provide hot reload, etc.
     -->
-    <application android:requestLegacyExternalStorage="true"/>
-
     <uses-permission android:name="android.permission.INTERNET"/>
-    <uses-permission android:name="android.permission.WRITE_EXTERNAL_STORAGE"/>
-    <uses-permission android:name="android.permission.READ_EXTERNAL_STORAGE"/>
 </manifest>

ui/flutter/android/app/src/main/AndroidManifest.xml

@@ -2,11 +2,8 @@
     package="com.gopeed">
 
     <uses-permission android:name="android.permission.INTERNET" />
-    <uses-permission android:name="android.permission.WRITE_EXTERNAL_STORAGE"/>
-    <uses-permission android:name="android.permission.READ_EXTERNAL_STORAGE"/>
 
     <application
-        android:requestLegacyExternalStorage="true"
         android:name="${applicationName}"
         android:icon="@mipmap/ic_launcher"
         android:label="Gopeed">

ui/flutter/android/app/src/profile/AndroidManifest.xml

@@ -4,9 +4,5 @@
          the Flutter tool needs it to communicate with the running application
          to allow setting breakpoints, to provide hot reload, etc.
     -->
-    <application android:requestLegacyExternalStorage="true"/>
-
     <uses-permission android:name="android.permission.INTERNET"/>
-    <uses-permission android:name="android.permission.WRITE_EXTERNAL_STORAGE"/>
-    <uses-permission android:name="android.permission.READ_EXTERNAL_STORAGE"/>
 </manifest>

ui/flutter/lib/pages/app/app_controller.dart

@@ -1,7 +1,6 @@
 import 'dart:convert';
 import 'dart:ui' as ui;
 
-import 'package:external_path/external_path.dart';
 import 'package:flutter/material.dart';
 import 'package:get/get.dart';
 import 'package:path_provider/path_provider.dart';
@@ -157,23 +156,16 @@ class AppController extends GetxController {
       extra.bt.trackerSubscribeUrls.addAll(allTrackerSubscribeUrls);
     }
 
-    if (Util.isAndroid()) {
+    if (Util.isDesktop()) {
-      config.downloadDir =
+      config.downloadDir = (await getDownloadsDirectory())?.path ?? "./";
-          '${await ExternalPath.getExternalStoragePublicDirectory(ExternalPath.DIRECTORY_DOWNLOADS)}/com.gopeed';
+    } else if (Util.isAndroid()) {
+      config.downloadDir = (await getExternalStorageDirectory())?.path ??
+          (await getApplicationDocumentsDirectory()).path;
       return;
-    }
+    } else if (Util.isIOS()) {
-    if (Util.isIOS()) {
+      config.downloadDir = (await getApplicationDocumentsDirectory()).path;
-      config.downloadDir = 'Documents';
+    } else {
-      return;
+      config.downloadDir = './';
-    }
-
-    if (config.downloadDir.isEmpty) {
-      if (Util.isWeb()) {
-        config.downloadDir = './';
-      }
-      if (Util.isDesktop()) {
-        config.downloadDir = (await getDownloadsDirectory())?.path ?? "./";
-      }
     }
   }
 

ui/flutter/lib/pages/app/app_view.dart

@@ -1,4 +1,5 @@
 import 'package:flutter/material.dart';
+import 'package:flutter_localizations/flutter_localizations.dart';
 import 'package:get/get.dart';
 
 import '../../i18n/messages.dart';
@@ -20,6 +21,12 @@ class AppView extends GetView<AppController> {
       translations: messages,
       locale: toLocale(config.extra.locale),
       fallbackLocale: fallbackLocale,
+      localizationsDelegates: const [
+        GlobalMaterialLocalizations.delegate,
+        GlobalWidgetsLocalizations.delegate,
+        GlobalCupertinoLocalizations.delegate,
+      ],
+      supportedLocales: messages.keys.keys.map((e) => toLocale(e)).toList(),
       getPages: Routes.routes,
     );
   }

ui/flutter/lib/pages/create/create_view.dart

@@ -1,7 +1,6 @@
 import 'package:desktop_drop/desktop_drop.dart';
 import 'package:flutter/material.dart';
 import 'package:get/get.dart';
-import 'package:permission_handler/permission_handler.dart';
 import 'package:rounded_loading_button/rounded_loading_button.dart';
 
 import '../../api/api.dart';
@@ -171,13 +170,15 @@ class CreateView extends GetView<CreateController> {
                       try {
                         if (createFormKey.currentState!.validate()) {
                           downloadController.start();
-                          if (Util.isAndroid()) {
+
-                            if (!await Permission.storage.request().isGranted) {
+                          // if (Util.isAndroid()) {
-                              Get.snackbar('error'.tr,
+                          //   if (!await Permission.storage.request().isGranted) {
-                                  'create.error.noStoragePermission'.tr);
+                          //     Get.snackbar('error'.tr,
-                              return;
+                          //         'create.error.noStoragePermission'.tr);
-                            }
+                          //     return;
-                          }
+                          //   }
+                          // }
+
                           await createTask(CreateTask(
                               res: res,
                               opts: Options(

ui/flutter/lib/pages/setting/setting_view.dart

@@ -135,15 +135,12 @@ class SettingView extends GetView<SettingController> {
                 controller: trackerUpdateController,
                 child: Text('setting.update'.tr),
               ),
-              Flexible(
+              Expanded(
-                child: SizedBox(
+                child: SwitchListTile(
-                  width: 200,
+                    controlAffinity: ListTileControlAffinity.leading,
-                  child: SwitchListTile(
+                    value: true,
-                      controlAffinity: ListTileControlAffinity.leading,
+                    onChanged: (bool value) {},
-                      value: true,
+                    title: Text('setting.updateDaily'.tr)),
-                      onChanged: (bool value) {},
-                      title: Text('setting.updateDaily'.tr)),
-                ),
               ),
             ],
           ),
@@ -234,7 +231,8 @@ class SettingView extends GetView<SettingController> {
           title: Text('setting.homepage'.tr),
           subtitle: const Text('https://github.com/monkeyWie/gopeed'),
           onTap: () {
-            launchUrl(Uri.parse('https://github.com/monkeyWie/gopeed'));
+            launchUrl(Uri.parse('https://github.com/monkeyWie/gopeed'),
+                mode: LaunchMode.externalApplication);
           },
         );
     buildVersion() => ListTile(

ui/flutter/lib/pages/task/task_view.dart

@@ -30,13 +30,14 @@ class Item extends StatelessWidget {
               ),
               Row(
                 children: [
-                  Expanded(
+                  SizedBox(
+                      width: 180,
                       child: Text(
-                    "${isDone() ? "" : "${Util.fmtByte(task.progress.downloaded)} / "}${Util.fmtByte(task.size)}",
+                        "${isDone() ? "" : "${Util.fmtByte(task.progress.downloaded)} / "}${Util.fmtByte(task.size)}",
-                    style: context.textTheme.bodyText1
+                        style: context.textTheme.bodyText1
-                        ?.copyWith(color: Get.theme.disabledColor),
+                            ?.copyWith(color: Get.theme.disabledColor),
-                  ).padding(left: 18)),
+                      ).padding(left: 18)),
-                  Expanded(
+                  Flexible(
                       child: Row(
                     mainAxisAlignment: MainAxisAlignment.end,
                     children: [

ui/flutter/pubspec.lock

@@ -197,13 +197,6 @@ packages:
       url: "https://pub.flutter-io.cn"
     source: hosted
     version: "4.0.6"
-  external_path:
-    dependency: "direct main"
-    description:
-      name: external_path
-      url: "https://pub.flutter-io.cn"
-    source: hosted
-    version: "1.0.3"
   fake_async:
     dependency: transitive
     description:
@@ -309,6 +302,11 @@ packages:
       url: "https://pub.flutter-io.cn"
     source: hosted
     version: "2.0.1"
+  flutter_localizations:
+    dependency: "direct main"
+    description: flutter
+    source: sdk
+    version: "0.0.0"
   flutter_test:
     dependency: "direct dev"
     description: flutter
@@ -564,41 +562,6 @@ packages:
       url: "https://pub.flutter-io.cn"
     source: hosted
     version: "2.1.3"
-  permission_handler:
-    dependency: "direct main"
-    description:
-      name: permission_handler
-      url: "https://pub.flutter-io.cn"
-    source: hosted
-    version: "10.1.0"
-  permission_handler_android:
-    dependency: transitive
-    description:
-      name: permission_handler_android
-      url: "https://pub.flutter-io.cn"
-    source: hosted
-    version: "10.1.0"
-  permission_handler_apple:
-    dependency: transitive
-    description:
-      name: permission_handler_apple
-      url: "https://pub.flutter-io.cn"
-    source: hosted
-    version: "9.0.6"
-  permission_handler_platform_interface:
-    dependency: transitive
-    description:
-      name: permission_handler_platform_interface
-      url: "https://pub.flutter-io.cn"
-    source: hosted
-    version: "3.9.0"
-  permission_handler_windows:
-    dependency: transitive
-    description:
-      name: permission_handler_windows
-      url: "https://pub.flutter-io.cn"
-    source: hosted
-    version: "0.1.1"
   petitparser:
     dependency: transitive
     description:

ui/flutter/pubspec.yaml

@@ -15,7 +15,7 @@ publish_to: "none" # Remove this line if you wish to publish to pub.dev
 # In iOS, build-name is used as CFBundleShortVersionString while build-number used as CFBundleVersion.
 # Read more about iOS versioning at
 # https://developer.apple.com/library/archive/documentation/General/Reference/InfoPlistKeyReference/Articles/CoreFoundationKeys.html
-version: 1.2.0+1
+version: 1.2.3+1
 
 environment:
   sdk: ">=2.17.6 <3.0.0"
@@ -29,6 +29,8 @@ environment:
 dependencies:
   flutter:
     sdk: flutter
+  flutter_localizations:
+    sdk: flutter  
 
   # The following adds the Cupertino Icons font to your application.
   # Use with the CupertinoIcons class for iOS style icons.
@@ -46,8 +48,6 @@ dependencies:
   rounded_loading_button: ^2.1.0
   url_launcher: ^6.1.6
   shared_preferences: ^2.0.15
-  permission_handler: ^10.1.0
-  external_path: ^1.0.3
   logger: ^1.1.0
   desktop_drop: ^0.3.3
   intl: ^0.17.0