X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?p=scpubgit%2FObject-Remote.git;a=blobdiff_plain;f=lib%2FObject%2FRemote%2FRole%2FLogForwarder.pm;h=d701b728f6ae0e6f28fb66a44801c7302e3791b8;hp=3c1ef6241c683e2cb4dc83290efef525b32f0fc0;hb=f048e6dfeaefce08546e91cc9d4bf79492d7ca0d;hpb=4a9fa1a5a329754549f998536e0e68f19d42d7ab diff --git a/lib/Object/Remote/Role/LogForwarder.pm b/lib/Object/Remote/Role/LogForwarder.pm index 3c1ef62..d701b72 100644 --- a/lib/Object/Remote/Role/LogForwarder.pm +++ b/lib/Object/Remote/Role/LogForwarder.pm @@ -1,58 +1,36 @@ package Object::Remote::Role::LogForwarder; -use Moo::Role; -use Object::Remote::Logging; -use Scalar::Util qw(weaken); +use Moo::Role; use Carp qw(cluck); -with 'Log::Contextual::Role::Router'; - -#TODO re-weaken router references when object::remote -#weak reference operation is figured out - -has child_routers => ( is => 'ro', required => 1, default => sub { [] } ); -has parent_router => ( is => 'rw', );#weak_ref => 1 ); - -sub BUILD { } - -after BUILD => sub { - my ($self) = @_; -# my $parent = $self->parent_router; -# return unless defined $parent ; -# $parent->add_child_router($self); +has _forward_destination => ( is => 'rw' ); +has enable_forward => ( is => 'rw', default => sub { 1 } ); +has _forward_stop => ( is => 'ro', required => 1, default => sub { {} } ); + +after _deliver_message => sub { + my ($self, $level, $generator, $args, $metadata) = @_; + my $package = $metadata->{package}; + my $destination = $self->_forward_destination; + our $reentrant; + + return unless $self->enable_forward; + return unless defined $destination; + return if $self->_forward_stop->{$package}; + + if (defined $reentrant) { + warn "log forwarding went reentrant. bottom: '$reentrant' top: '$package'"; + return; + } + + local $reentrant = $package; + + $destination->_deliver_message($level, $generator, $args, $metadata); }; -sub describe { - my ($self, $depth) = @_; - $depth = -1 unless defined $depth; - $depth++; - my $buf = "\t" x $depth . $self->description . "\n"; - foreach my $child (@{$self->child_routers}) { - next unless defined $child; - $buf .= $child->describe($depth); - } - - return $buf; +sub exclude_forwarding { + my ($self, $package) = @_; + $package = caller unless defined $package; + $self->_forward_stop->{$package} = 1; } -sub add_child_router { - my ($self, $router) = @_; - push(@{ $self->child_routers }, $router); -# weaken(${ $self->child_routers }[-1]); - return; -} - -#sub remove_child_router { -# my ($self, $description) = @_; -# return delete $self->child_routers->{$description}; -#} - -after handle_log_message => sub { - my ($self, @args) = @_; - my $parent = $self->parent_router; - - return unless defined $parent; - $parent->handle_log_message(@args); -}; - 1;