tweak _get_loggers so it can tolerate a logger that is not aware of the log level

[scpubgit/Object-Remote.git] / lib / Object / Remote / Logging.pm

diff --git a/lib/Object/Remote/Logging.pm b/lib/Object/Remote/Logging.pm
index 25831ac..f765db8 100644 (file)
--- a/lib/Object/Remote/Logging.pm
+++ b/lib/Object/Remote/Logging.pm
@@ -1,15 +1,49 @@
 package Object::Remote::Logging;
 
-use strictures 1;
+use Moo;
+use Scalar::Util qw(blessed);
+use Object::Remote::Logging::Logger;
 
-use Log::Contextual qw( :log );
-use Object::Remote::LogRouter;
+extends 'Log::Contextual';
 
-use base qw(Log::Contextual); 
+sub router {
+   our $Router_Instance ||= do {
+      require Object::Remote::Logging::Router;
+      Object::Remote::Logging::Router->new;
+   }
+}
 
-sub arg_router { return $_[1] if defined $_[1]; our $Router_Instance ||= Object::Remote::LogRouter->new }
+sub arg_levels {
+    return [qw( trace debug verbose info warn error )];
+}
 
-sub init_node { my $n = `hostname`; chomp($n); $_[0]->add_child_router("[node $n]", __PACKAGE__->arg_router) }
+#this is invoked on all nodes
+sub init_logging {
+    my $level = $ENV{OBJECT_REMOTE_LOG_LEVEL};
+    return unless defined $level;
+    my $logger = Object::Remote::Logging::Logger->new(
+      min_level => lc($level),
+      level_names => Object::Remote::Logging::arg_levels(),
+    );
+    
+    #TODO check on speed of string compare against a hash with a single key
+    router()->connect(sub { 
+        return unless $_[1]->{controller} eq __PACKAGE__;
+        #skip things from remote hosts because they log to STDERR
+        #when OBJECT_REMOTE_LOG_LEVEL is in effect
+        return if $_[1]->{remote}->{connection_id};
+        $logger
+    });
+}
+
+#this is invoked by the controlling node
+#on the remote nodes
+sub init_logging_forwarding {
+  my ($self, %controller_info) = @_;
+  
+  router()->_remote_metadata({ connection_id => $controller_info{connection_id} });
+  router()->_forward_destination($controller_info{router});
+}
 
 1;