search

symfony / monolog-bundle

Symfony Monolog Bundle
symfony.com
MIT License
2.89k stars 231 forks source link

Add priority field to processor tags #455

Closed sudo-plz closed 3 weeks ago

sudo-plz commented 1 year ago

So that the order in which the processors are used by monolog can be determined. Example config:

Foo\Bar\Log\Processors\FirstProcessor:
    tags:
        - { name: monolog.processor, channel: http }

Foo\Bar\Log\Processors\SecondProcessor:
    tags:
        - { name: monolog.processor, channel: http, priority: -10 } # last

Foo\Bar\Log\Processors\ThirdProcessor:
    tags:
        - { name: monolog.processor, channel: http, priority: -20 } # very last

(PR in favor of old PR)

sudo-plz commented 1 year ago

Related PR has been merged, can you finish the review @HypeMC? Thanks!

nicklog commented 10 months ago

Hey, what's the status here? I'm also interested in the feature. Thanks :)

sudo-plz commented 10 months ago

Hey, what's the status here? I'm also interested in the feature. Thanks :)

Following comment is still open: https://github.com/symfony/monolog-bundle/pull/455/files#r1369858964 I didn't take the time to fix it anymore, however I'll check if I can implement it sometime (or feel free to change it)

alexndlm commented 3 months ago

@stof, @HypeMC, would you please check?

alexndlm commented 3 months ago

@Seldaek could you merge this?

alexndlm commented 3 months ago

@nicolas-grekas could you help to merge this?

stof commented 3 weeks ago

Closing in favor of #485 which took over that work