Merge pull request #96 from IamMuuo/local-auth

fix: local auth requiring password even on desktop and web
IamMuuo · Oct 3, 2024 · 43ae0db · 43ae0db
2 parents 22be4f1 + cadc35e
commit 43ae0db
Show file tree

Hide file tree

Showing 3 changed files with 18 additions and 37 deletions.
diff --git a/lib/controllers/controller_ready_mixin.dart b/lib/controllers/controller_ready_mixin.dart
diff --git a/lib/controllers/controllers.dart b/lib/controllers/controllers.dart
@@ -4,4 +4,3 @@ export 'network_controller.dart';
 export 'reward_controller.dart';
 export 'settings_controller.dart';
 export 'notifications_controller.dart';
-export 'controller_ready_mixin.dart';
diff --git a/lib/controllers/settings_controller.dart b/lib/controllers/settings_controller.dart
@@ -31,22 +31,25 @@ class SettingsController extends GetxController with StateMixin {
   Future<bool> performLocalAuthentication(String reason) async {
     bool authenticated = false;
 
-    try {
-      authenticated = await localAuthentication.authenticate(
-        localizedReason: reason,
-        options: const AuthenticationOptions(
-          stickyAuth: true,
-          sensitiveTransaction: true,
-        ),
-      );
-    } on PlatformException catch (e) {
-      authenticated = false;
-      debugPrint(e.toString());
-    } catch (e) {
-      debugPrint(e.toString());
-    }
+    if (Platform.isIOS || Platform.isAndroid) {
+      try {
+        authenticated = await localAuthentication.authenticate(
+          localizedReason: reason,
+          options: const AuthenticationOptions(
+            stickyAuth: true,
+            sensitiveTransaction: true,
+          ),
+        );
+      } on PlatformException catch (e) {
+        authenticated = false;
+        debugPrint(e.toString());
+      } catch (e) {
+        debugPrint(e.toString());
+      }
 
-    return authenticated;
+      return authenticated;
+    }
+    return true;
   }
 
   Future<bool> deviceSupportsLocalAuth() async {