X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?p=scpubgit%2FObject-Remote.git;a=blobdiff_plain;f=lib%2FObject%2FRemote%2FLogging%2FRouter.pm;h=14b97add48fb1abbc124098987d7a783faef8890;hp=e73f00250a54fc88da7eeb634119f4adc3382419;hb=f4a8508058f7bad5052775a8df3e683527c73333;hpb=b49a063d7885c23278585e52c3066c791f8934f2 diff --git a/lib/Object/Remote/Logging/Router.pm b/lib/Object/Remote/Logging/Router.pm index e73f002..14b97ad 100644 --- a/lib/Object/Remote/Logging/Router.pm +++ b/lib/Object/Remote/Logging/Router.pm @@ -12,73 +12,73 @@ has _connections => ( is => 'ro', required => 1, default => sub { [] } ); has _remote_metadata => ( is => 'rw' ); sub before_import { - my ($self, $controller, $importer, $spec) = @_; + my ($self, $controller, $importer, $spec) = @_; } sub after_import { } sub _get_loggers { - my ($self, %metadata) = @_; - my $package = $metadata{package}; - my $level = $metadata{level}; - my $is_level = "is_$level"; - my $need_clean = 0; - my @loggers; - - foreach my $selector (@{$self->_connections}) { - unless(defined $selector) { - $need_clean = 1; - next; - } - - foreach my $logger ($selector->($package, { %metadata })) { - next unless defined $logger; - my $method = $logger->can($is_level); - next unless defined $method; - next unless $logger->$method; - push(@loggers, $logger); - } + my ($self, %metadata) = @_; + my $package = $metadata{package}; + my $level = $metadata{level}; + my $is_level = "is_$level"; + my $need_clean = 0; + my @loggers; + + foreach my $selector (@{$self->_connections}) { + unless(defined $selector) { + $need_clean = 1; + next; + } + + foreach my $logger ($selector->($package, { %metadata })) { + next unless defined $logger; + my $method = $logger->can($is_level); + next unless defined $method; + next unless $logger->$method; + push(@loggers, $logger); } + } - $self->_clean_connections if $need_clean; - - return @loggers; + $self->_clean_connections if $need_clean; + + return @loggers; } sub handle_log_request { - my ($self, $metadata_in, $generator, @args) = @_; - my %metadata = %{$metadata_in}; - my $level = $metadata{level}; - my $package = $metadata{package}; - my $need_clean = 0; - - #caller_level is useless when log forwarding is in place - #so we won't tempt people with using it for now - access - #to caller level will be available in the future - delete $metadata{caller_level}; - $metadata{object_remote} = $self->_remote_metadata; - - foreach my $logger ($self->_get_loggers(%metadata)) { - $logger->$level([ $generator->(@args) ], \%metadata); - } + my ($self, $metadata_in, $generator, @args) = @_; + my %metadata = %{$metadata_in}; + my $level = $metadata{level}; + my $package = $metadata{package}; + my $need_clean = 0; + + #caller_level is useless when log forwarding is in place + #so we won't tempt people with using it for now - access + #to caller level will be available in the future + delete $metadata{caller_level}; + $metadata{object_remote} = $self->_remote_metadata; + $metadata{timestamp} = time; + foreach my $logger ($self->_get_loggers(%metadata)) { + $logger->$level([ $generator->(@args) ], \%metadata); + } } sub connect { - my ($self, $destination) = @_; - my $wrapped; - - if (ref($destination) ne 'CODE') { - $wrapped = sub { $destination }; - } else { - $wrapped = $destination; - } - - push(@{$self->_connections}, $wrapped); + my ($self, $destination) = @_; + my $wrapped; + + if (ref($destination) ne 'CODE') { + $wrapped = sub { $destination }; + } else { + $wrapped = $destination; + } + + push(@{$self->_connections}, $wrapped); } sub _clean_connections { - my ($self) = @_; - @{$self->{_connections}} = grep { defined } @{$self->{_connections}}; + my ($self) = @_; + @{$self->{_connections}} = grep { defined } @{$self->{_connections}}; } 1;