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;fp=lib%2FObject%2FRemote%2FRole%2FLogForwarder.pm;h=d701b728f6ae0e6f28fb66a44801c7302e3791b8;hp=a6ba93976cab77cce07d7553db14afd364b99b26;hb=f048e6dfeaefce08546e91cc9d4bf79492d7ca0d;hpb=ae19820130feb4fc48d8e365ce5c3109642d203c diff --git a/lib/Object/Remote/Role/LogForwarder.pm b/lib/Object/Remote/Role/LogForwarder.pm index a6ba939..d701b72 100644 --- a/lib/Object/Remote/Role/LogForwarder.pm +++ b/lib/Object/Remote/Role/LogForwarder.pm @@ -7,29 +7,24 @@ has _forward_destination => ( is => 'rw' ); has enable_forward => ( is => 'rw', default => sub { 1 } ); has _forward_stop => ( is => 'ro', required => 1, default => sub { {} } ); -around _get_loggers => sub { - my ($orig, $self, %metadata) = @_; - my $package = $metadata{package}; - my %clone = %metadata; +after _deliver_message => sub { + my ($self, $level, $generator, $args, $metadata) = @_; + my $package = $metadata->{package}; + my $destination = $self->_forward_destination; our $reentrant; - - return if $reentrant; - local($reentrant) = 1; - - my @loggers = $orig->($self, %clone); - if (! $self->enable_forward || $self->_forward_stop->{$package}) { - #warn "will not forward log events for '$package'"; - return @loggers; + 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; } - my $forward_to = $self->_forward_destination; - - if ($forward_to) { - push(@loggers, $forward_to->_get_loggers(%clone)); - } + local $reentrant = $package; - return @loggers; + $destination->_deliver_message($level, $generator, $args, $metadata); }; sub exclude_forwarding {