Move RGBController base destructor functionality into RGBController::Shutdown() so that update thread can be stopped before deleting the controller. Shutdown() must be called in every RGBController implementation before deleting the controller. Also some fixes to the NetworkServer to avoid deadlocking and disconnect issues

Controllers/NanoleafController/RGBController_Nanoleaf.cpp

@@ -87,6 +87,11 @@ RGBController_Nanoleaf::RGBController_Nanoleaf(std::string a_address, int a_port
     SetupZones();
 }
 
+RGBController_Nanoleaf::~RGBController_Nanoleaf()
+{
+    Shutdown();
+}
+
 void RGBController_Nanoleaf::SetupZones()
 {
     zone led_zone;

Controllers/NanoleafController/RGBController_Nanoleaf.h

@@ -19,6 +19,7 @@ class RGBController_Nanoleaf : public RGBController
 
 public:
     RGBController_Nanoleaf(std::string a_address, int a_port, std::string a_auth_token);
+    ~RGBController_Nanoleaf();
 
     void        SetupZones();