feat(logger): allow overriding STDOUT levels

Author: stakach

shard.yml

@@ -1,5 +1,5 @@
 name: placeos-driver
-version: 7.18.3
+version: 7.19.0
 
 dependencies:
   action-controller:

src/placeos-driver/driver_manager.cr

@@ -4,7 +4,9 @@ require "ipaddress"
 class PlaceOS::Driver::DriverManager
   def initialize(@module_id : String, @model : DriverModel, logger_io = STDOUT, subscriptions = nil, edge_driver = false)
     @settings = Settings.new @model.settings
-    @logger = PlaceOS::Driver::Log.new(@module_id, logger_io)
+    log_level = (@settings.get { setting?(::Log::Severity, :persistent_log_level) } || ::Log::Severity::Error) rescue ::Log::Severity::Error
+
+    @logger = PlaceOS::Driver::Log.new(@module_id, logger_io, severity: log_level)
     @queue = Queue.new(@logger) { |state| connection(state) }
     @schedule = PlaceOS::Driver::Proxy::Scheduler.new(@logger)
     @subscriptions = edge_driver ? nil : Proxy::Subscriptions.new(subscriptions || Subscriptions.new, @logger)
@@ -153,6 +155,10 @@ class PlaceOS::Driver::DriverManager
     driver.on_update
   rescue error
     logger.error(exception: error) { "during settings update of #{@driver.class} (#{@module_id})" }
+  ensure
+    # set any new persistent log levels
+    log_level = (@settings.get { setting?(::Log::Severity, :persistent_log_level) } || ::Log::Severity::Error) rescue ::Log::Severity::Error
+    @logger.override_io_severity(log_level)
   end
 
   def execute(json)

src/placeos-driver/logger.cr

@@ -52,14 +52,22 @@ class PlaceOS::Driver
     getter io_backend : ::Log::IOBackend
     getter protocol_backend : ProtocolBackend
     getter debugging : Bool
+    getter io_severity : ::Log::Severity
 
     def debugging=(value : Bool)
       @debugging = value
 
       # Don't worry it's not really an append, it's updating a hash with the
       # backend as the key, so this is a clean update
       @broadcast_backend.append(@protocol_backend, value ? ::Log::Severity::Debug : ::Log::Severity::None)
-      self.level = value ? ::Log::Severity::Debug : ::Log::Severity::Error
+      self.level = value ? ::Log::Severity::Debug : @io_severity
+    end
+
+    def override_io_severity(severity : ::Log::Severity)
+      @io_severity = severity
+      # Don't worry it's not really an append, it's updating a hash
+      @broadcast_backend.append(@io_backend, severity)
+      self.level = @debugging ? ::Log::Severity::Debug : severity
     end
 
     def initialize(
@@ -69,6 +77,7 @@ class PlaceOS::Driver
       severity : ::Log::Severity = ::Log::Severity::Error,
     )
       @debugging = false
+      @io_severity = severity
 
       # Create a Driver protocol log backend
       @protocol_backend = ProtocolBackend.new(protocol: @protocol)