feat(logging): integrate Sentry logging support for all platforms

2026-08-31 04:32:32 +00:00 · 2022-09-01 15:51:54 -04:00
parent 3593eb329b
commit 57bbb80a01
17 changed files with 113 additions and 17 deletions
--- a/.github/workflows/prepare.yml
+++ b/.github/workflows/prepare.yml
@@ -27,6 +27,8 @@ on:
    secrets:
      FIREBASE_TOKEN:
        required: true
+      SENTRY_DSN:
+        required: true

 jobs:
  build-details:
@@ -131,6 +133,7 @@ jobs:
          export FLAVOR=${{ needs.build-details.outputs.build-flavor }}
          export COMMIT=${{ github.sha }}
          export BUILD=${{ needs.build-details.outputs.build-number }}
+          export SENTRY_DSN=${{ secrets.SENTRY_DSN }}
          flutter packages pub run environment_config:generate
     
      - name: Generate Localization
--- a/.github/workflows/release.yml
+++ b/.github/workflows/release.yml
@@ -25,6 +25,7 @@ jobs:
      flavor: ${{ inputs.flavor }}
    secrets:
      FIREBASE_TOKEN: ${{ secrets.FIREBASE_TOKEN }}
+      SENTRY_DSN: ${{ secrets.SENTRY_DSN }}

  build-android:
    name: Build Android
--- a/environment_config.yaml
+++ b/environment_config.yaml
@@ -15,3 +15,8 @@ environment_config:
      type: String
      default: edge
      env_var: FLAVOR
+    sentryDSN:
+      type: String
+      default: ""
+      env_var: SENTRY_DSN
+
--- a/ios/Podfile.lock
+++ b/ios/Podfile.lock
@@ -58,7 +58,7 @@ PODS:
  - firebase_core (1.21.1):
    - Firebase/CoreOnly (= 9.4.0)
    - Flutter
-  - firebase_messaging (12.0.1):
+  - firebase_messaging (12.0.3):
    - Firebase/Messaging (= 9.4.0)
    - firebase_core
    - Flutter
@@ -163,6 +163,13 @@ PODS:
  - SDWebImage (5.13.2):
    - SDWebImage/Core (= 5.13.2)
  - SDWebImage/Core (5.13.2)
+  - Sentry (7.23.0):
+    - Sentry/Core (= 7.23.0)
+  - Sentry/Core (7.23.0)
+  - sentry_flutter (0.0.1):
+    - Flutter
+    - FlutterMacOS
+    - Sentry (~> 7.23.0)
  - share_plus (0.0.1):
    - Flutter
  - shared_preferences_ios (0.0.1):
@@ -189,6 +196,7 @@ DEPENDENCIES:
  - package_info_plus (from `.symlinks/plugins/package_info_plus/ios`)
  - path_provider_ios (from `.symlinks/plugins/path_provider_ios/ios`)
  - quick_actions_ios (from `.symlinks/plugins/quick_actions_ios/ios`)
+  - sentry_flutter (from `.symlinks/plugins/sentry_flutter/ios`)
  - share_plus (from `.symlinks/plugins/share_plus/ios`)
  - shared_preferences_ios (from `.symlinks/plugins/shared_preferences_ios/ios`)
  - sqflite (from `.symlinks/plugins/sqflite/ios`)
@@ -217,6 +225,7 @@ SPEC REPOS:
    - nanopb
    - PromisesObjC
    - SDWebImage
+    - Sentry
    - SwiftyGif

 EXTERNAL SOURCES:
@@ -247,6 +256,8 @@ EXTERNAL SOURCES:
    :path: ".symlinks/plugins/path_provider_ios/ios"
  quick_actions_ios:
    :path: ".symlinks/plugins/quick_actions_ios/ios"
+  sentry_flutter:
+    :path: ".symlinks/plugins/sentry_flutter/ios"
  share_plus:
    :path: ".symlinks/plugins/share_plus/ios"
  shared_preferences_ios:
@@ -269,7 +280,7 @@ SPEC CHECKSUMS:
  Firebase: 7703fc4022824b6d6db1bf7bea58d13b8e17ec46
  firebase_auth: 74cfbf7be1e3e7fdaac9e7f117cefd89155251a9
  firebase_core: 8e500850dd138c269d197c8dfca3ae886fa3b6f2
-  firebase_messaging: 2a3412ef5e30b84adfb66ae358e7a8fdebcd7462
+  firebase_messaging: b65dacd4de1b469893dea0d754b1947f45ef4f7a
  firebase_storage: 846832b7f2d22238678dc9abc513f2151f102462
  FirebaseAppCheckInterop: 8e95b4329809e3c06fb425acb3839dcadfe23673
  FirebaseAuth: ae5d4402e1516497357d909162b091b3ca2a2e9c
@@ -296,6 +307,8 @@ SPEC CHECKSUMS:
  PromisesObjC: ab77feca74fa2823e7af4249b8326368e61014cb
  quick_actions_ios: 5ec8f5f1ae81512ac803fe10c197ebb875767a9e
  SDWebImage: 72f86271a6f3139cc7e4a89220946489d4b9a866
+  Sentry: a0d4563fa4ddacba31fdcc35daaa8573d87224d6
+  sentry_flutter: 8bde7d0e57a721727fe573f13bb292c497b5a249
  share_plus: 056a1e8ac890df3e33cb503afffaf1e9b4fbae68
  shared_preferences_ios: 548a61f8053b9b8a49ac19c1ffbc8b92c50d68ad
  sqflite: 6d358c025f5b867b29ed92fc697fd34924e11904
--- a/ios/Runner.xcodeproj/project.pbxproj
+++ b/ios/Runner.xcodeproj/project.pbxproj
@@ -424,6 +424,7 @@
 				"${BUILT_PRODUCTS_DIR}/GoogleUtilities/GoogleUtilities.framework",
 				"${BUILT_PRODUCTS_DIR}/PromisesObjC/FBLPromises.framework",
 				"${BUILT_PRODUCTS_DIR}/SDWebImage/SDWebImage.framework",
+				"${BUILT_PRODUCTS_DIR}/Sentry/Sentry.framework",
 				"${BUILT_PRODUCTS_DIR}/SwiftyGif/SwiftyGif.framework",
 				"${BUILT_PRODUCTS_DIR}/file_picker/file_picker.framework",
 				"${BUILT_PRODUCTS_DIR}/flutter_native_splash/flutter_native_splash.framework",
@@ -432,6 +433,7 @@
 				"${BUILT_PRODUCTS_DIR}/package_info_plus/package_info_plus.framework",
 				"${BUILT_PRODUCTS_DIR}/path_provider_ios/path_provider_ios.framework",
 				"${BUILT_PRODUCTS_DIR}/quick_actions_ios/quick_actions_ios.framework",
+				"${BUILT_PRODUCTS_DIR}/sentry_flutter/sentry_flutter.framework",
 				"${BUILT_PRODUCTS_DIR}/share_plus/share_plus.framework",
 				"${BUILT_PRODUCTS_DIR}/shared_preferences_ios/shared_preferences_ios.framework",
 				"${BUILT_PRODUCTS_DIR}/sqflite/sqflite.framework",
@@ -458,6 +460,7 @@
 				"${TARGET_BUILD_DIR}/${FRAMEWORKS_FOLDER_PATH}/GoogleUtilities.framework",
 				"${TARGET_BUILD_DIR}/${FRAMEWORKS_FOLDER_PATH}/FBLPromises.framework",
 				"${TARGET_BUILD_DIR}/${FRAMEWORKS_FOLDER_PATH}/SDWebImage.framework",
+				"${TARGET_BUILD_DIR}/${FRAMEWORKS_FOLDER_PATH}/Sentry.framework",
 				"${TARGET_BUILD_DIR}/${FRAMEWORKS_FOLDER_PATH}/SwiftyGif.framework",
 				"${TARGET_BUILD_DIR}/${FRAMEWORKS_FOLDER_PATH}/file_picker.framework",
 				"${TARGET_BUILD_DIR}/${FRAMEWORKS_FOLDER_PATH}/flutter_native_splash.framework",
@@ -466,6 +469,7 @@
 				"${TARGET_BUILD_DIR}/${FRAMEWORKS_FOLDER_PATH}/package_info_plus.framework",
 				"${TARGET_BUILD_DIR}/${FRAMEWORKS_FOLDER_PATH}/path_provider_ios.framework",
 				"${TARGET_BUILD_DIR}/${FRAMEWORKS_FOLDER_PATH}/quick_actions_ios.framework",
+				"${TARGET_BUILD_DIR}/${FRAMEWORKS_FOLDER_PATH}/sentry_flutter.framework",
 				"${TARGET_BUILD_DIR}/${FRAMEWORKS_FOLDER_PATH}/share_plus.framework",
 				"${TARGET_BUILD_DIR}/${FRAMEWORKS_FOLDER_PATH}/shared_preferences_ios.framework",
 				"${TARGET_BUILD_DIR}/${FRAMEWORKS_FOLDER_PATH}/sqflite.framework",
--- a/lib/main.dart
+++ b/lib/main.dart
@@ -10,6 +10,7 @@ import 'package:lunasea/system/cache/image/image_cache.dart';
 import 'package:lunasea/system/in_app_purchase/in_app_purchase.dart';
 import 'package:lunasea/system/localization.dart';
 import 'package:lunasea/system/network/network.dart';
+import 'package:lunasea/system/sentry.dart';
 import 'package:lunasea/system/window_manager/window_manager.dart';
 import 'package:lunasea/system/platform.dart';

@@ -21,9 +22,10 @@ Future<void> main() async {
  runZonedGuarded(
    () async {
      //LunaSea initialization
+      await LunaSentry().initialize();
      await LunaDatabase().initialize();
-      if (LunaFirebase.isSupported) await LunaFirebase().initialize();
      LunaLogger().initialize();
+      if (LunaFirebase.isSupported) await LunaFirebase().initialize();
      LunaTheme().initialize();
      if (LunaWindowManager.isSupported) await LunaWindowManager().initialize();
      if (LunaNetwork.isSupported) LunaNetwork().initialize();
--- a/lib/router/router.dart
+++ b/lib/router/router.dart
@@ -1,5 +1,6 @@
 import 'package:flutter/foundation.dart';
 import 'package:lunasea/system/logger.dart';
+import 'package:lunasea/system/sentry.dart';
 import 'package:lunasea/widgets/pages/error_route.dart';
 import 'package:lunasea/router/routes.dart';
 import 'package:lunasea/vendor.dart';
@@ -10,11 +11,10 @@ class LunaRouter {
  void initialize() {
    router = GoRouter(
      debugLogDiagnostics: kDebugMode,
+      errorBuilder: (_, state) => ErrorRoutePage(exception: state.error),
      initialLocation: LunaRoutes.initialLocation,
+      observers: [LunaSentry().navigatorObserver],
      routes: LunaRoutes.values.map((r) => r.root.routes).toList(),
-      errorBuilder: (_, state) {
-        return ErrorRoutePage(exception: state.error);
-      },
    );
  }

--- a/lib/system/logger.dart
+++ b/lib/system/logger.dart
@@ -2,6 +2,7 @@ import 'package:flutter/foundation.dart';
 import 'package:flutter/material.dart';
 import 'package:lunasea/core.dart';
 import 'package:lunasea/database/models/log.dart';
+import 'package:lunasea/system/sentry.dart';
 import 'package:lunasea/types/exception.dart';
 import 'package:lunasea/types/log_type.dart';

@@ -53,34 +54,44 @@ class LunaLogger {
  }

  void error(String message, dynamic error, StackTrace? stackTrace) {
+    if (kDebugMode) {
+      print(message);
+      print(error);
+      print(stackTrace);
+    }
+
    if (error is! NetworkImageLoadException) {
+      if (error is! DioError) {
+        LunaSentry().captureException(error, stackTrace);
+      }
+
      LunaLog log = LunaLog.withError(
        type: LunaLogType.ERROR,
        message: message,
        error: error,
        stackTrace: stackTrace,
      );
-      if (kDebugMode) {
-        print(message);
-        print(error);
-        print(stackTrace);
-      }
      LunaBox.logs.create(log);
    }
  }

  void critical(dynamic error, StackTrace stackTrace) {
+    if (kDebugMode) {
+      print(error);
+      print(stackTrace);
+    }
+
    if (error is! NetworkImageLoadException) {
+      if (error is! DioError) {
+        LunaSentry().captureException(error, stackTrace);
+      }
+
      LunaLog log = LunaLog.withError(
        type: LunaLogType.CRITICAL,
        message: error?.toString() ?? LunaUI.TEXT_EMDASH,
        error: error,
        stackTrace: stackTrace,
      );
-      if (kDebugMode) {
-        print(error);
-        print(stackTrace);
-      }
      LunaBox.logs.create(log);
    }
  }
--- a/lib/system/sentry.dart
+++ b/lib/system/sentry.dart
@@ -0,0 +1,18 @@
+import 'package:lunasea/system/environment.dart';
+import 'package:sentry_flutter/sentry_flutter.dart';
+
+class LunaSentry {
+  Future<void> initialize() async {
+    await SentryFlutter.init((options) {
+      options.dsn = LunaEnvironment.sentryDSN;
+      options.environment = LunaEnvironment.flavor;
+      options.release = LunaEnvironment.commit;
+    });
+  }
+
+  Future<void> captureException(dynamic error, StackTrace? stackTrace) async {
+    await Sentry.captureException(error, stackTrace: stackTrace);
+  }
+
+  SentryNavigatorObserver get navigatorObserver => SentryNavigatorObserver();
+}
--- a/linux/flutter/generated_plugin_registrant.cc
+++ b/linux/flutter/generated_plugin_registrant.cc
@@ -7,6 +7,7 @@
 #include "generated_plugin_registrant.h"

 #include <screen_retriever/screen_retriever_plugin.h>
+#include <sentry_flutter/sentry_flutter_plugin.h>
 #include <url_launcher_linux/url_launcher_plugin.h>
 #include <window_manager/window_manager_plugin.h>

@@ -14,6 +15,9 @@ void fl_register_plugins(FlPluginRegistry* registry) {
  g_autoptr(FlPluginRegistrar) screen_retriever_registrar =
      fl_plugin_registry_get_registrar_for_plugin(registry, "ScreenRetrieverPlugin");
  screen_retriever_plugin_register_with_registrar(screen_retriever_registrar);
+  g_autoptr(FlPluginRegistrar) sentry_flutter_registrar =
+      fl_plugin_registry_get_registrar_for_plugin(registry, "SentryFlutterPlugin");
+  sentry_flutter_plugin_register_with_registrar(sentry_flutter_registrar);
  g_autoptr(FlPluginRegistrar) url_launcher_linux_registrar =
      fl_plugin_registry_get_registrar_for_plugin(registry, "UrlLauncherPlugin");
  url_launcher_plugin_register_with_registrar(url_launcher_linux_registrar);
--- a/linux/flutter/generated_plugins.cmake
+++ b/linux/flutter/generated_plugins.cmake
@@ -4,6 +4,7 @@

 list(APPEND FLUTTER_PLUGIN_LIST
  screen_retriever
+  sentry_flutter
  url_launcher_linux
  window_manager
 )
--- a/macos/Flutter/GeneratedPluginRegistrant.swift
+++ b/macos/Flutter/GeneratedPluginRegistrant.swift
@@ -13,6 +13,7 @@ import firebase_storage
 import package_info_plus_macos
 import path_provider_macos
 import screen_retriever
+import sentry_flutter
 import share_plus_macos
 import shared_preferences_macos
 import sqflite
@@ -28,6 +29,7 @@ func RegisterGeneratedPlugins(registry: FlutterPluginRegistry) {
  FLTPackageInfoPlusPlugin.register(with: registry.registrar(forPlugin: "FLTPackageInfoPlusPlugin"))
  PathProviderPlugin.register(with: registry.registrar(forPlugin: "PathProviderPlugin"))
  ScreenRetrieverPlugin.register(with: registry.registrar(forPlugin: "ScreenRetrieverPlugin"))
+  SentryFlutterPlugin.register(with: registry.registrar(forPlugin: "SentryFlutterPlugin"))
  SharePlusMacosPlugin.register(with: registry.registrar(forPlugin: "SharePlusMacosPlugin"))
  SharedPreferencesPlugin.register(with: registry.registrar(forPlugin: "SharedPreferencesPlugin"))
  SqflitePlugin.register(with: registry.registrar(forPlugin: "SqflitePlugin"))
--- a/macos/Podfile.lock
+++ b/macos/Podfile.lock
@@ -26,7 +26,7 @@ PODS:
  - firebase_core (1.21.1):
    - Firebase/CoreOnly (~> 9.4.0)
    - FlutterMacOS
-  - firebase_messaging (12.0.1):
+  - firebase_messaging (12.0.3):
    - Firebase/CoreOnly (~> 9.4.0)
    - Firebase/Messaging (~> 9.4.0)
    - firebase_core
@@ -126,6 +126,13 @@ PODS:
  - PromisesObjC (2.1.1)
  - screen_retriever (0.0.1):
    - FlutterMacOS
+  - Sentry (7.23.0):
+    - Sentry/Core (= 7.23.0)
+  - Sentry/Core (7.23.0)
+  - sentry_flutter (0.0.1):
+    - Flutter
+    - FlutterMacOS
+    - Sentry (~> 7.23.0)
  - share_plus_macos (0.0.1):
    - FlutterMacOS
  - shared_preferences_macos (0.0.1):
@@ -149,6 +156,7 @@ DEPENDENCIES:
  - package_info_plus_macos (from `Flutter/ephemeral/.symlinks/plugins/package_info_plus_macos/macos`)
  - path_provider_macos (from `Flutter/ephemeral/.symlinks/plugins/path_provider_macos/macos`)
  - screen_retriever (from `Flutter/ephemeral/.symlinks/plugins/screen_retriever/macos`)
+  - sentry_flutter (from `Flutter/ephemeral/.symlinks/plugins/sentry_flutter/macos`)
  - share_plus_macos (from `Flutter/ephemeral/.symlinks/plugins/share_plus_macos/macos`)
  - shared_preferences_macos (from `Flutter/ephemeral/.symlinks/plugins/shared_preferences_macos/macos`)
  - sqflite (from `Flutter/ephemeral/.symlinks/plugins/sqflite/macos`)
@@ -175,6 +183,7 @@ SPEC REPOS:
    - GTMSessionFetcher
    - nanopb
    - PromisesObjC
+    - Sentry

 EXTERNAL SOURCES:
  cloud_firestore:
@@ -198,6 +207,8 @@ EXTERNAL SOURCES:
    :path: Flutter/ephemeral/.symlinks/plugins/path_provider_macos/macos
  screen_retriever:
    :path: Flutter/ephemeral/.symlinks/plugins/screen_retriever/macos
+  sentry_flutter:
+    :path: Flutter/ephemeral/.symlinks/plugins/sentry_flutter/macos
  share_plus_macos:
    :path: Flutter/ephemeral/.symlinks/plugins/share_plus_macos/macos
  shared_preferences_macos:
@@ -219,7 +230,7 @@ SPEC CHECKSUMS:
  Firebase: 7703fc4022824b6d6db1bf7bea58d13b8e17ec46
  firebase_auth: d73121234b93a92fa130c1595c59afd37cbfcce8
  firebase_core: 822a1076483bf9764284322c9310daa98e1e6817
-  firebase_messaging: 564a55c225276ba69412e29557c8d280916aa22f
+  firebase_messaging: b76079f26d7a22470b2de423198a0cabf98ebf03
  firebase_storage: 5e4de184fe773a22f4c65875d1e71c009a4cc7b0
  FirebaseAppCheckInterop: 8e95b4329809e3c06fb425acb3839dcadfe23673
  FirebaseAuth: ae5d4402e1516497357d909162b091b3ca2a2e9c
@@ -243,6 +254,8 @@ SPEC CHECKSUMS:
  path_provider_macos: 3c0c3b4b0d4a76d2bf989a913c2de869c5641a19
  PromisesObjC: ab77feca74fa2823e7af4249b8326368e61014cb
  screen_retriever: 59634572a57080243dd1bf715e55b6c54f241a38
+  Sentry: a0d4563fa4ddacba31fdcc35daaa8573d87224d6
+  sentry_flutter: 8bde7d0e57a721727fe573f13bb292c497b5a249
  share_plus_macos: 853ee48e7dce06b633998ca0735d482dd671ade4
  shared_preferences_macos: a64dc611287ed6cbe28fd1297898db1336975727
  sqflite: a5789cceda41d54d23f31d6de539d65bb14100ea
--- a/pubspec.lock
+++ b/pubspec.lock
@@ -1029,6 +1029,20 @@ packages:
      url: "https://pub.dartlang.org"
    source: hosted
    version: "0.1.2"
+  sentry:
+    dependency: transitive
+    description:
+      name: sentry
+      url: "https://pub.dartlang.org"
+    source: hosted
+    version: "6.9.1"
+  sentry_flutter:
+    dependency: "direct main"
+    description:
+      name: sentry_flutter
+      url: "https://pub.dartlang.org"
+    source: hosted
+    version: "6.9.1"
  share_plus:
    dependency: "direct main"
    description:
--- a/pubspec.yaml
+++ b/pubspec.yaml
@@ -44,6 +44,7 @@ dependencies:
  percent_indicator: ^4.2.2
  quick_actions: ^1.0.0
  retrofit: ^3.0.1+1
+  sentry_flutter: ^6.9.1
  share_plus: ^4.0.10+1
  shimmer: ^2.0.0
  simple_icons: ^7.1.0
--- a/windows/flutter/generated_plugin_registrant.cc
+++ b/windows/flutter/generated_plugin_registrant.cc
@@ -7,12 +7,15 @@
 #include "generated_plugin_registrant.h"

 #include <screen_retriever/screen_retriever_plugin.h>
+#include <sentry_flutter/sentry_flutter_plugin.h>
 #include <url_launcher_windows/url_launcher_windows.h>
 #include <window_manager/window_manager_plugin.h>

 void RegisterPlugins(flutter::PluginRegistry* registry) {
  ScreenRetrieverPluginRegisterWithRegistrar(
      registry->GetRegistrarForPlugin("ScreenRetrieverPlugin"));
+  SentryFlutterPluginRegisterWithRegistrar(
+      registry->GetRegistrarForPlugin("SentryFlutterPlugin"));
  UrlLauncherWindowsRegisterWithRegistrar(
      registry->GetRegistrarForPlugin("UrlLauncherWindows"));
  WindowManagerPluginRegisterWithRegistrar(
--- a/windows/flutter/generated_plugins.cmake
+++ b/windows/flutter/generated_plugins.cmake
@@ -4,6 +4,7 @@

 list(APPEND FLUTTER_PLUGIN_LIST
  screen_retriever
+  sentry_flutter
  url_launcher_windows
  window_manager
 )