NewRelic integration (#717)

* NewRelic integration - iOS

* NewRelic integration - Android

* BackgroundLogger removed

* iOS pipeline - macOS version bumped

* Proguard rules updated && notification builder disposed fix

Author: akrol95

BMM.Api/Framework/ILogger.cs

@@ -9,5 +9,6 @@ public interface ILogger
         void Warn(string tag, string message);
         void Error(string tag, string message);
         void Error(string tag, string message, Exception exception, bool presentedToUser = false);
+        void TrackEvent(string message, IDictionary<string, string> properties);
     }
 }

BMM.Core/App.cs

@@ -79,7 +79,6 @@
 using BMM.Core.Models.POs.Tracks.Interfaces;
 using BMM.Core.NewMediaPlayer;
 using BMM.Core.NewMediaPlayer.Abstractions;
-using Microsoft.AppCenter;
 using MvvmCross;
 using MvvmCross.Base;
 using MvvmCross.IoC;
@@ -115,10 +114,6 @@ public override void Initialize()
             Registrations.Start(GlobalConstants.ApplicationName);
             BlobCache.ApplicationName = GlobalConstants.PackageName;
 
-#if DEBUG
-            AppCenter.LogLevel = LogLevel.Info;
-#endif
-
             SetupLanguageBinder();
 
             Mvx.IoCProvider.LazyConstructAndRegisterSingleton<IRememberedQueueService, RememberedQueueService>();
@@ -134,7 +129,7 @@ public override void Initialize()
             Mvx.IoCProvider.RegisterType<IResponseDeserializer, ResponseDeserializer>();
             Mvx.IoCProvider.RegisterType<IBmmVersionProvider, BmmVersionProvider>();
 
-            Mvx.IoCProvider.RegisterType<IAnalytics, AppCenterAnalytics>();
+            Mvx.IoCProvider.RegisterType<IAnalytics, NewRelicAnalytics>();
             Mvx.IoCProvider.LazyConstructAndRegisterSingleton<IDeepLinkHandler, DeepLinkHandler>();
             Mvx.IoCProvider.RegisterType<IShareLink, ShareLink>();
             Mvx.IoCProvider.RegisterType<IAppNavigator, AppNavigator>();
@@ -277,8 +272,6 @@ public override void Initialize()
                     },
                     MvxReference.Strong);
 
-            Mvx.IoCProvider.ConstructAndRegisterSingleton<BackgroundLogger, BackgroundLogger>();
-
             Mvx.IoCProvider.RegisterType<PersistedEventWriter>();
             Mvx.IoCProvider.RegisterType<FirebaseConfigUpdater>();
             Mvx.IoCProvider.RegisterType<AfterStartupSupportEndsPopupDisplayer>();

BMM.Core/BMM.Core.csproj

@@ -16,8 +16,6 @@
       <PackageReference Include="akavache" Version="9.1.7" />
       <PackageReference Include="IdentityModel" Version="7.0.0" />
       <PackageReference Include="IdentityModel.OidcClient" Version="6.0.0" />
-      <PackageReference Include="Microsoft.AppCenter" Version="5.0.1" />
-      <PackageReference Include="Microsoft.AppCenter.Analytics" Version="5.0.1" />
       <PackageReference Include="MvvmCross" Version="9.2.0" />
       <PackageReference Include="MvvmCross.Plugin.JsonLocalization" Version="9.2.0" />
       <PackageReference Include="MvvmCross.Plugin.Messenger" Version="9.2.0" />

BMM.Core/Constants/AnalyticsConstants.cs

@@ -2,6 +2,7 @@ namespace BMM.Core.Constants
 {
     public class AnalyticsConstants
     {
+        public const string NewRelicEventType = "BMM";
         public const string ConnectionParameterName = "ConnectionType";
     }
 }

BMM.Core/Helpers/AnalyticsInitializer.cs

@@ -1,6 +1,4 @@
-using Microsoft.AppCenter;
-using Microsoft.AppCenter.Analytics;
-using Microsoft.Maui.Devices;
+using Microsoft.Maui.Devices;
 
 namespace BMM.Core.Helpers
 {
@@ -15,9 +13,6 @@ public abstract class AnalyticsInitializer
 
         public static void IOSRegister()
         {
-            if (!AppCenter.Configured)
-                AppCenter.Start(GlobalConstants.iOSAppSecret, typeof(Analytics));
-                    
             if (ShouldInitSentry)
                 SentrySdk.Init(SetupSentry);
         }

BMM.Core/Helpers/GlobalConstants.cs

@@ -56,9 +56,9 @@ public class GlobalConstants
         public const bool PushNotificationsEnabledDefault = true;
         public const int NetworkRequestTimeout = 10;
         public const int DefaultNumberOfPodcastTracksToDownload = 3;
-        public const string DroidAppSecret = "#{Android_APP_SECRET_PLACEHOLDER}#";
-        public const string iOSAppSecret = "#{iOS_APP_SECRET_PLACEHOLDER}#";
         public const string SentryDsn = "#{SENTRY_DSN_PLACEHOLDER}#";
+        public const string NewRelicAndroidToken = "#{NEW_RELIC_ANDROID_PLACEHOLDER}#";
+        public const string NewRelic_iOSToken = "#{NEW_RELIC_IOS_PLACEHOLDER}#";
 
         public const string AndroidUpdateLink = "https://play.google.com/store/apps/details?id=org.brunstad.bmm";
         public const string IosUpdateLink = "itms-apps://apps.apple.com/no/app/bmm-brunstad/id777577855";

…ntations/Analytics/AppCenterAnalytics.cs …entations/Analytics/NewRelicAnalytics.csBMM.Core/Implementations/Analytics/AppCenterAnalytics.cs renamed to BMM.Core/Implementations/Analytics/NewRelicAnalytics.cs BMM.Core/Implementations/Analytics/AppCenterAnalytics.cs renamed to BMM.Core/Implementations/Analytics/NewRelicAnalytics.cs

@@ -8,14 +8,14 @@
 
 namespace BMM.Core.Implementations.Analytics
 {
-    public class AppCenterAnalytics : IAnalytics
+    public class NewRelicAnalytics : IAnalytics
     {
         private readonly ILogger _logger;
         private readonly IUserStorage _userStorage;
         private readonly IFirebaseRemoteConfig _remoteConfig;
         private readonly IConnection _connection;
 
-        public AppCenterAnalytics(
+        public NewRelicAnalytics(
             ILogger logger,
             IUserStorage userStorage,
             IFirebaseRemoteConfig remoteConfig,
@@ -48,9 +48,9 @@ public void LogEvent(string eventName, IDictionary<string, object> parameters)
 
             parameters.Add(AnalyticsConstants.ConnectionParameterName, AnalyticsUtils.GetConnectionType(_connection));
 
-            Dictionary<string, string> dString = parameters.ToDictionary(k => k.Key, k => k.Value == null ? "null" : k.Value.ToString());
+            var dString = parameters.ToDictionary(k => k.Key, k => k.Value == null ? "null" : k.Value.ToString());
             _logger.Info("Analytics", eventName + " {" + string.Join(",", dString.Select(kv => $"{kv.Key}={kv.Value}")) + "}");
-            Microsoft.AppCenter.Analytics.Analytics.TrackEvent(eventName, dString);
+            _logger.TrackEvent(eventName, dString);
         }
     }
 }

BMM.Core/Implementations/Downloading/BackgroundLogger.cs

@@ -44,5 +44,10 @@ public void Error(string tag, string message, Exception exception, bool presente
             _logger.Error(tag, message, exception, presentedToUser);
             _mainThreadAsyncDispatcher.ExecuteOnMainThreadAsync(() => _userDialogs.AlertAsync(exception.Message));
         }
+
+        public void TrackEvent(string message, IDictionary<string, string> properties)
+        {
+            _logger.TrackEvent(message, properties);
+        }
     }
 }

BMM.Core/Implementations/Exceptions/ErrorDialogDisplayingLogger.cs

@@ -1,9 +1,7 @@
 using BMM.Api.Framework;
 using BMM.Core.Constants;
-using BMM.Core.Implementations.Exceptions;
 using BMM.Core.Implementations.Security;
 using BMM.Core.Utils;
-using Microsoft.AppCenter;
 
 namespace BMM.Core.Implementations.Logger
 {
@@ -25,22 +23,24 @@ protected BaseLogger(
             _connection = connection;
             _userStorage = userStorage;
         }
-        
+
         public void Debug(string tag, string message)
         {
-            AppCenterLog.Debug(tag, message);
+            Console.WriteLine($"DEBUG - {tag} - {message}");
         }
 
         public void Info(string tag, string message)
         {
-            AppCenterLog.Info(tag, message);
+            Console.WriteLine($"INFO - {tag} - {message}");
         }
 
+        public abstract void TrackEvent(string message, IDictionary<string, string> properties);
+
         public void Warn(string tag, string message)
         {
             var parameters = InitializeDictionaryWithBasicParameters(tag, message);
-            AppCenterLog.Warn(tag, message);
-            Microsoft.AppCenter.Analytics.Analytics.TrackEvent("Warning", parameters);
+            Console.WriteLine($"WARN - {tag} - {message}");
+            TrackEvent("Warning", parameters);
         }
 
         public virtual void Error(string tag, string message)
@@ -51,21 +51,20 @@ public virtual void Error(string tag, string message)
                 GetSentryScope(parameters),
                 SentryLevel.Error);
 
-            AppCenterLog.Error(tag, message);
-            Microsoft.AppCenter.Analytics.Analytics.TrackEvent("Error without exception", parameters);
+            TrackEvent("Error without exception", parameters);
         }
 
         public virtual void Error(string tag, string message, Exception exception, bool presentedToUser = false)
         {
             var parameters = InitializeDictionaryWithBasicParameters(tag, message, presentedToUser);
 
             SentrySdk.CaptureException(exception, GetSentryScope(parameters));
-            AppCenterLog.Error(tag, message, exception);
-                       
+            Console.WriteLine($"ERROR - {tag} - {message} \n \n {exception}");
+         
             parameters.Add(StackTrackParameterName, exception.StackTrace);
             parameters.Add(ExceptionParameterName, exception.ToString());
 
-            Microsoft.AppCenter.Analytics.Analytics.TrackEvent("Error with exception", parameters);
+           TrackEvent("Error with exception", parameters);
         }
 
         private static Action<Scope> GetSentryScope(IDictionary<string, string> parameters)