Skip to content

Commit

Permalink
chore: remove useless handler arg
Browse files Browse the repository at this point in the history
  • Loading branch information
cdaguerre committed Jun 30, 2024
1 parent cee36ca commit d8edcee
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 3 deletions.
1 change: 0 additions & 1 deletion src/DependencyInjection/config/tracing/tracing.php
Original file line number Diff line number Diff line change
Expand Up @@ -94,7 +94,6 @@

->set(TracingHandler::class)
->args([
service(TracerProviderInterface::class),
service(MainSpanContextInterface::class),
param('tracing.logs.level'),
param('tracing.logs.channels'),
Expand Down
3 changes: 1 addition & 2 deletions src/Tracing/Instrumentation/LogHandler/TracingHandler.php
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,6 @@
use Monolog\Handler\AbstractProcessingHandler;
use Monolog\Logger;
use Monolog\Processor\PsrLogMessageProcessor;
use OpenTelemetry\API\Trace\TracerProviderInterface;
use OpenTelemetry\SDK\Trace\Span;

class TracingHandler extends AbstractProcessingHandler
Expand All @@ -29,7 +28,7 @@ class TracingHandler extends AbstractProcessingHandler
/**
* @param array<string> $channels
*/
public function __construct(protected TracerProviderInterface $tracerProvider, protected MainSpanContextInterface $mainSpanContext, $level = Logger::INFO, private array $channels = [], private string $strategy = self::STRATEGY_MAIN_SPAN, bool $bubble = true)
public function __construct(protected MainSpanContextInterface $mainSpanContext, $level = Logger::INFO, private array $channels = [], private string $strategy = self::STRATEGY_MAIN_SPAN, bool $bubble = true)
{
parent::__construct($level, $bubble);

Expand Down

0 comments on commit d8edcee

Please sign in to comment.