Merge pull request #101 from doingnz/Remove-UdpLogger-when-network-disconnects

Remove UdpLogger when network disconnects

Author: adrianstevens

Source/Meadow.Clima/Controllers/NetworkController.cs

@@ -39,7 +39,25 @@ public class NetworkController
     /// Gets the period for triggering network down events.
     /// </summary>
     public TimeSpan DownEventPeriod { get; } = TimeSpan.FromSeconds(30);
-    
+
+
+    /// <summary>
+    /// Port used for UdpLogging. 
+    /// </summary>
+    /// <remarks>
+    /// Default set in constructor is port 5100
+    /// </remarks>
+    private int UdpLoggingPort { get; set; }
+
+    /// <summary>
+    /// Instance of UdpLogger. Use to remove UdpLogger if the network disconnects
+    /// </summary>
+    private UdpLogger? UdpLogger { get; set; }
+
+    private string WifiSsid { get; set; } = "SSID";
+    private string WifiPassword { get; set; } = "PASSWORD";
+
+
     /// <summary>
     /// Initializes a new instance of the <see cref="NetworkController"/> class.
     /// </summary>
@@ -74,8 +92,8 @@ public async Task<bool> ConnectToCloud()
         {
             if (!wifi.IsConnected)
             {
-                Resolver.Log.Info("Connecting to network...");
-                await wifi.Connect("interwebs", "1234567890");
+                Resolver.Log.Info($"Connecting to network: {WifiSsid}");
+                await wifi.Connect(WifiSsid, WifiPassword);
             }
         }
 
@@ -119,6 +137,14 @@ private void DownEventTimerProc(object _)
 
     private void OnNetworkDisconnected(INetworkAdapter sender, NetworkDisconnectionEventArgs args)
     {
+        // Remove the UdpLogger if it's in the LogProviderCollection.
+        if (UdpLogger != null)
+        {
+            Resolver.Log.RemoveProvider(UdpLogger);
+            UdpLogger.Dispose();
+            UdpLogger = null;
+        }
+
         lastDown = DateTimeOffset.UtcNow;
         downEventTimer.Change(DownEventPeriod, TimeSpan.FromMilliseconds(-1));
         ConnectionStateChanged?.Invoke(this, false);
@@ -152,7 +178,7 @@ private async Task ReportWiFiScan(IWiFiNetworkAdapter wifi)
     private void OnNetworkConnected(INetworkAdapter sender, NetworkConnectionEventArgs args)
     {
         Resolver.Log.Info("Add UdpLogger");
-        Resolver.Log.AddProvider(new UdpLogger(/*port = */5100));
+        Resolver.Log.AddProvider(UdpLogger = new UdpLogger(UdpLoggingPort));
 
         if (sender is IWiFiNetworkAdapter wifi)
         {