Dealing with logging issues.

Author: std-s

android/gradle.properties

@@ -1,3 +1,7 @@
-org.gradle.jvmargs=-Xmx4816M
+org.gradle.jvmargs=-Xmx6144M
+org.gradle.parallel=true
+org.gradle.configuration-cache=true
+org.gradle.daemon=true
+
 android.useAndroidX=true
 android.enableJetifier=true

lib/app.dart

@@ -19,7 +19,6 @@ class ChatApp extends StatelessWidget {
         debugShowCheckedModeBanner: false,
         enableLog: true,
         builder: builder,
-        logWriterCallback: Logger.print,
         translations: TranslationService(),
         localizationsDelegates: const [
           GlobalMaterialLocalizations.delegate,

lib/core/controller/app_controller.dart

@@ -80,19 +80,6 @@ class AppController extends GetxController with UpgradeManger {
     super.onInit();
   }
 
-  void _requestPermissions() {
-    flutterLocalNotificationsPlugin
-        .resolvePlatformSpecificImplementation<AndroidFlutterLocalNotificationsPlugin>()
-        ?.requestNotificationsPermission();
-    flutterLocalNotificationsPlugin
-        .resolvePlatformSpecificImplementation<IOSFlutterLocalNotificationsPlugin>()
-        ?.requestPermissions(
-          alert: true,
-          badge: true,
-          sound: true,
-        );
-  }
-
   Future<void> showNotification(im.Message message, {bool showNotification = true}) async {
     if (_isGlobalNotDisturb() ||
         message.attachedInfoElem?.notSenderNotificationPush == true ||
@@ -124,11 +111,11 @@ class AppController extends GetxController with UpgradeManger {
       if (Platform.isAndroid) {
         final id = seq;
 
-        const androidPlatformChannelSpecifics = AndroidNotificationDetails('chat', 'OpenIM聊天消息',
-            channelDescription: '来自OpenIM的信息', importance: Importance.max, priority: Priority.high, ticker: 'ticker');
+        const androidPlatformChannelSpecifics = AndroidNotificationDetails('chat', 'OpenIM Chat',
+            channelDescription: 'OpenIM Chat Message', importance: Importance.max, priority: Priority.high, ticker: 'ticker');
         const NotificationDetails platformChannelSpecifics =
             NotificationDetails(android: androidPlatformChannelSpecifics);
-        await flutterLocalNotificationsPlugin.show(id, '您收到了一条新消息', '消息内容：.....', platformChannelSpecifics,
+        await flutterLocalNotificationsPlugin.show(id, 'You have a new message', 'Message：.....', platformChannelSpecifics,
             payload: '');
       }
     }
@@ -137,24 +124,6 @@ class AppController extends GetxController with UpgradeManger {
   Future<void> _cancelAllNotifications() async {
     await flutterLocalNotificationsPlugin.cancelAll();
   }
-
-  Future<void> _startForegroundService() async {
-    await getAppInfo();
-    const androidPlatformChannelSpecifics = AndroidNotificationDetails('pro', 'OpenIM后台进程',
-        channelDescription: '保证app能收到信息', importance: Importance.max, priority: Priority.high, ticker: 'ticker');
-
-    await flutterLocalNotificationsPlugin
-        .resolvePlatformSpecificImplementation<AndroidFlutterLocalNotificationsPlugin>()
-        ?.startForegroundService(1, packageInfo!.appName, '正在运行...',
-            notificationDetails: androidPlatformChannelSpecifics, payload: '');
-  }
-
-  Future<void> _stopForegroundService() async {
-    await flutterLocalNotificationsPlugin
-        .resolvePlatformSpecificImplementation<AndroidFlutterLocalNotificationsPlugin>()
-        ?.stopForegroundService();
-  }
-
   void showBadge(count) {
     OpenIM.iMManager.messageManager.setAppBadge(count);
 

lib/core/controller/im_controller.dart

@@ -158,6 +158,7 @@ class IMController extends GetxController with IMCallback, OpenIMLive {
         onJoinedGroupDeleted: joinedGroupDeleted,
       ));
 
+    Logger().sdkIsInited = initialized;
     initializedSubject.sink.add(initialized);
   }
 
@@ -168,7 +169,6 @@ class IMController extends GetxController with IMCallback, OpenIMLive {
         token: token,
         defaultValue: () async => UserInfo(userID: userID),
       );
-      ApiService().setToken(token);
       userInfo = UserFullInfo.fromJson(user.toJson()).obs;
       _queryMyFullInfo();
       _queryAtAllTag();

lib/main.dart

@@ -14,6 +14,6 @@ void main() {
 
     Config.init(() => runApp(const ChatApp()));
   }, (error, stackTrace) {
-    Logger.print('FlutterError: ${error.toString()}, ${stackTrace.toString()}');
+    Logger.print('FlutterError: ${error.toString()}, ${stackTrace.toString()}', onlyConsole: true);
   });
 }

lib/pages/splash/splash_logic.dart

@@ -21,7 +21,6 @@ class SplashLogic extends GetxController {
   @override
   void onInit() {
     initializedSub = imLogic.initializedSubject.listen((value) {
-      Logger.print('---------------------initialized---------------------');
       if (null != userID && null != token) {
         _login();
       } else {

openim_common/lib/openim_common.dart

@@ -30,7 +30,6 @@ export 'src/utils/permissions.dart';
 export 'src/utils/sp_util.dart';
 export 'src/utils/utils.dart';
 export 'src/utils/voice_record.dart';
-export 'src/utils/api_service.dart';
 export 'src/widgets/avatar_view.dart';
 export 'src/widgets/azlist_view.dart';
 export 'src/widgets/bottom_bar.dart';

openim_common/lib/src/apis.dart

@@ -1,14 +1,8 @@
 import 'dart:async';
-import 'dart:convert';
-
-import 'package:collection/collection.dart';
 import 'package:dio/dio.dart';
 import 'package:flutter_openim_sdk/flutter_openim_sdk.dart';
 import 'package:get/get.dart';
 import 'package:openim_common/openim_common.dart';
-import 'package:sprintf/sprintf.dart';
-
-import 'utils/api_service.dart';
 
 class Apis {
   static Options get imTokenOptions => Options(headers: {'token': DataSp.imToken});
@@ -42,21 +36,11 @@ class Apis {
         'verifyCode': verificationCode,
       });
       final cert = LoginCertificate.fromJson(data!);
-      ApiService().setToken(cert.imToken);
 
       return cert;
-    } catch (e, _) {
-      final t = e as (int, String?)?;
-
-      if (t == null) {
-        Logger.print('e:$e');
+    } catch (e, s) {
+      _catchErrorHelper(e, s);
 
-        return Future.error(e);
-      }
-      final errCode = t.$1;
-      final errMsg = t.$2;
-      _kickoff(errCode);
-      Logger.print('e:$errCode s:$errMsg');
       return Future.error(e);
     }
   }
@@ -95,15 +79,11 @@ class Apis {
       });
 
       final cert = LoginCertificate.fromJson(data!);
-      ApiService().setToken(cert.imToken);
 
       return cert;
     } catch (e, s) {
-      final t = e as (int, String?);
-      final errCode = t.$1;
-      final errMsg = t.$2;
-      _kickoff(errCode);
-      Logger.print('e:$errCode s:$errMsg');
+      _catchErrorHelper(e, s);
+
       return Future.error(e);
     }
   }
@@ -129,11 +109,7 @@ class Apis {
         options: chatTokenOptions,
       );
     } catch (e, s) {
-      final t = e as (int, String?);
-      final errCode = t.$1;
-      final errMsg = t.$2;
-      _kickoff(errCode);
-      Logger.print('e:$errCode s:$errMsg');
+      _catchErrorHelper(e, s);
     }
   }
 
@@ -155,11 +131,8 @@ class Apis {
       );
       return true;
     } catch (e, s) {
-      final t = e as (int, String?);
-      final errCode = t.$1;
-      final errMsg = t.$2;
-      _kickoff(errCode);
-      Logger.print('e:$errCode s:$errMsg');
+      _catchErrorHelper(e, s);
+
       return false;
     }
   }
@@ -228,18 +201,15 @@ class Apis {
         options: chatTokenOptions,
       );
     } catch (e, s) {
-      final t = e as (int, String?);
-      final errCode = t.$1;
-      final errMsg = t.$2;
-      _kickoff(errCode);
-      Logger.print('e:$errCode s:$errMsg');
+      _catchErrorHelper(e, s);
     }
   }
 
   static Future<List<FriendInfo>> searchFriendInfo(
     String keyword, {
     int pageNumber = 1,
     int showNumber = 10,
+    bool showErrorToast = true,
   }) async {
     try {
       final data = await HttpUtil.post(
@@ -249,18 +219,16 @@ class Apis {
           'keyword': keyword,
         },
         options: chatTokenOptions,
+        showErrorToast: showErrorToast,
       );
       if (data['users'] is List) {
         return (data['users'] as List).map((e) => FriendInfo.fromJson(e)).toList();
       }
       return [];
     } catch (e, s) {
-      final t = e as (int, String?);
-      final errCode = t.$1;
-      final errMsg = t.$2;
-      _kickoff(errCode);
-      Logger.print('e:$errCode s:$errMsg');
-      return [];
+      _catchErrorHelper(e, s);
+
+      rethrow;
     }
   }
 
@@ -284,11 +252,8 @@ class Apis {
       }
       return null;
     } catch (e, s) {
-      final t = e as (int, String?);
-      final errCode = t.$1;
-      final errMsg = t.$2;
-      _kickoff(errCode);
-      Logger.print('e:$errCode s:$errMsg');
+      _catchErrorHelper(e, s);
+
       return [];
     }
   }
@@ -312,11 +277,8 @@ class Apis {
       }
       return null;
     } catch (e, s) {
-      final t = e as (int, String?);
-      final errCode = t.$1;
-      final errMsg = t.$2;
-      _kickoff(errCode);
-      Logger.print('e:$errCode s:$errMsg');
+      _catchErrorHelper(e, s);
+
       return [];
     }
   }
@@ -348,7 +310,8 @@ class Apis {
       IMViews.showToast(StrRes.sentSuccessfully);
       return true;
     }).catchError((e, s) {
-      Logger.print('e:$e s:$s');
+      _catchErrorHelper(e, s);
+
       return false;
     });
   }
@@ -365,7 +328,9 @@ class Apis {
       final signaling = SignalingCertificate.fromJson(value)..roomID = roomID;
       return signaling;
     }).catchError((e, s) {
-      Logger.print('e:$e s:$s');
+      _catchErrorHelper(e, s);
+
+      throw e;
     });
   }
 
@@ -413,23 +378,24 @@ class Apis {
     return {'discoverPageURL': Config.discoverPageURL, 'allowSendMsgNotFriend': Config.allowSendMsgNotFriend};
   }
 
-  static void _catchError(Object e, StackTrace s, {bool forceBack = true}) {
-    if (e is ApiException) {
-      var msg = '${e.code}'.tr;
-      if (msg.isEmpty || e.code.toString() == msg) {
-        msg = e.message ?? 'Unkonw error';
-      } else if (e.code == 1004) {
-        msg = sprintf(msg, [StrRes.meeting]);
-      }
-
-      IMViews.showToast(msg);
+  static void _catchErrorHelper(Object e, StackTrace s) {
+    if (e is (int, String?)) {
+      final errCode = e.$1;
+      final errMsg = e.$2;
+      _kickoff(errCode);
 
-      if ((e.code == 10010 || e.code == 10002) && forceBack) {
-        DataSp.removeLoginCertificate();
-        Get.offAllNamed('/login');
-      }
+      Logger.print('e:$errCode s:$errMsg');
     } else {
-      IMViews.showToast(e.toString());
+      _catchError(e, s);
+    }
+  }
+
+  static void _catchError(Object e, StackTrace s, {bool forceBack = true}) {
+    IMViews.showToast(e.toString());
+
+    if (forceBack) {
+      DataSp.removeLoginCertificate();
+      Get.offAllNamed('/login');
     }
   }
 }

openim_common/lib/src/config.dart

@@ -6,7 +6,6 @@ import 'package:flutter_openim_sdk/flutter_openim_sdk.dart';
 import 'package:hive_flutter/hive_flutter.dart';
 import 'package:media_kit/media_kit.dart';
 import 'package:openim_common/openim_common.dart';
-import 'package:openim_common/src/utils/api_service.dart';
 import 'package:package_info_plus/package_info_plus.dart';
 import 'package:path_provider/path_provider.dart';
 
@@ -20,7 +19,6 @@ class Config {
       await Hive.initFlutter(path);
       MediaKit.ensureInitialized();
       HttpUtil.init();
-      ApiService().setBaseUrl(serverIp);
     } catch (_) {}
 
     runApp();
@@ -65,7 +63,7 @@ class Config {
   static const friendScheme = "io.openim.app/addFriend/";
   static const groupScheme = "io.openim.app/joinGroup/";
 
-  static const _host = "116.205.175.233";
+  static const _host = "your-server-ip or your-domain";
 
   static const _ipRegex = '((2[0-4]\\d|25[0-5]|[01]?\\d\\d?)\\.){3}(2[0-4]\\d|25[0-5]|[01]?\\d\\d?)';