[GH-ISSUE #1378] Level is not being read from config for custom handler #576

New issue

Closed

opened 2026-03-04 02:16:06 +03:00 by kerem · 1 comment

kerem commented

2026-03-04 02:16:06 +03:00

Owner

Originally created by @mattdillon100 on GitHub (Sep 13, 2019).
Original GitHub issue: https://github.com/Seldaek/monolog/issues/1378

I am using Monolog version 1 with Symfony.

I'm not sure that this should be asked here or in the Symfony repo but will ask here first.

I have created a custom handler to push logs to AWS Firehose. Here is the constructor:

class FirehoseLogHandler extends AbstractProcessingHandler {

public function __construct(
        FirehoseClient $firehoseClient,
        FormatterInterface $formatter,
        $streamName,
        $level = Logger::INFO,
        $bubble = true
    ) {

        $this->firehoseClient = $firehoseClient;
        $this->streamName = $streamName;
        $this->formatter = $formatter;

        parent::__construct($level, $bubble);
    }

And here is my monolog.yaml config:

monolog:
  handlers:
    firehose_handler:
      type: service
      id: kinesis_stream_handler

    main:
      type: stream
      handler: firehose_handler
      level: error
      channels: ["!event"]

services:
  kinesis_stream_handler:
    class: App\Logger\Handler\FirehoseLogHandler
    arguments:
      $firehoseClient: '@aws.firehose'
      $formatter: '@App\Logger\Formatter\FirehoseLogFormatter'
      $streamName: 'firehose-stream-test'

The problem that I am having is that the $level is always set to Logger::INFO (200), or whatever is set in the constructor. It seems to be ignoring what is set in the yaml config

Originally created by @mattdillon100 on GitHub (Sep 13, 2019). Original GitHub issue: https://github.com/Seldaek/monolog/issues/1378 I am using Monolog version 1 with Symfony. I'm not sure that this should be asked here or in the Symfony repo but will ask here first. I have created a custom handler to push logs to AWS Firehose. Here is the constructor: ``` class FirehoseLogHandler extends AbstractProcessingHandler { public function __construct( FirehoseClient $firehoseClient, FormatterInterface $formatter, $streamName, $level = Logger::INFO, $bubble = true ) { $this->firehoseClient = $firehoseClient; $this->streamName = $streamName; $this->formatter = $formatter; parent::__construct($level, $bubble); } ``` And here is my `monolog.yaml` config: ``` monolog: handlers: firehose_handler: type: service id: kinesis_stream_handler main: type: stream handler: firehose_handler level: error channels: ["!event"] services: kinesis_stream_handler: class: App\Logger\Handler\FirehoseLogHandler arguments: $firehoseClient: '@aws.firehose' $formatter: '@App\Logger\Formatter\FirehoseLogFormatter' $streamName: 'firehose-stream-test' ``` The problem that I am having is that the `$level` is always set to `Logger::INFO` (200), or whatever is set in the constructor. It seems to be ignoring what is set in the yaml config

kerem

2026-03-04 02:16:06 +03:00

closed this issue
added the
Support
label

kerem commented

2026-03-04 02:16:07 +03:00

Author

Owner

@alexandrmazur96 commented on GitHub (Sep 13, 2019):

Hi!

I think it happens because you do not pass the level parameter to firehose_handler.

As Symfony docs say:

This defines a stack of handlers and each handler is called in the order that it's defined.

But I can be wrong with this. Try to add level: error:

monolog:
  handlers:
    firehose_handler:
      type: service
      id: kinesis_stream_handler
      level: error

p.s. yes, this question should be to Symfony, not for Monolog. I guess.

@alexandrmazur96 commented on GitHub (Sep 13, 2019): Hi! I think it happens because you do not pass the `level` parameter to `firehose_handler`. As Symfony docs say: > This defines a stack of handlers and each handler is called in the order that it's defined. But I can be wrong with this. Try to add `level: error`: ``` monolog: handlers: firehose_handler: type: service id: kinesis_stream_handler level: error ``` p.s. yes, this question should be to Symfony, not for Monolog. I guess.

kerem referenced this issue

2026-03-04 03:07:25 +03:00

[PR #576] [CLOSED] Fixed an error in elastica requirements (missing comma) #1240