Skip to content

Commit 558d30d

Browse files
HypeMCHypeMC
committed
Add handler_default_channels configuration option
1 parent 6201959 commit 558d30d

File tree

7 files changed

+188
-7
lines changed

7 files changed

+188
-7
lines changed

‎config/schema/monolog-1.0.xsd‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@
1111
<xsd:choice minOccurs="0" maxOccurs="unbounded">
1212
<xsd:element name="handler" type="handler" />
1313
<xsd:element name="channel" type="xsd:string" />
14+
<xsd:element name="handler-default-channels" type="channels" minOccurs="0" maxOccurs="1" />
1415
</xsd:choice>
1516
</xsd:complexType>
1617

@@ -93,6 +94,7 @@
9394
<xsd:attribute name="webhook-url" type="xsd:string" />
9495
<xsd:attribute name="slack-team" type="xsd:string" />
9596
<xsd:attribute name="region" type="xsd:string" />
97+
<xsd:attribute name="use-default-channels" type="xsd:boolean" />
9698
</xsd:complexType>
9799

98100
<xsd:simpleType name="level">

‎src/DependencyInjection/Configuration.php‎

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -390,6 +390,8 @@ public function getConfigTreeBuilder(): TreeBuilder
390390
$treeBuilder = new TreeBuilder('monolog');
391391
$rootNode = $treeBuilder->getRootNode();
392392

393+
$this->addChannelsSection($rootNode, 'handler_default_channels');
394+
393395
$handlers = $rootNode
394396
->fixXmlConfig('channel')
395397
->fixXmlConfig('handler')
@@ -634,6 +636,7 @@ public function getConfigTreeBuilder(): TreeBuilder
634636
->end()
635637
->scalarNode('formatter')->end()
636638
->booleanNode('nested')->defaultFalse()->end()
639+
->booleanNode('use_default_channels')->defaultTrue()->end()
637640
->end();
638641

639642
$this->addGelfSection($handlerNode);
@@ -1086,11 +1089,11 @@ private function addVerbosityLevelSection(ArrayNodeDefinition $handlerNode)
10861089
;
10871090
}
10881091

1089-
private function addChannelsSection(ArrayNodeDefinition $handlerNode)
1092+
private function addChannelsSection(ArrayNodeDefinition $handlerNode, string $nodeName = 'channels')
10901093
{
10911094
$handlerNode
10921095
->children()
1093-
->arrayNode('channels')
1096+
->arrayNode($nodeName)
10941097
->fixXmlConfig('channel', 'elements')
10951098
->canBeUnset()
10961099
->beforeNormalization()
@@ -1106,7 +1109,7 @@ private function addChannelsSection(ArrayNodeDefinition $handlerNode)
11061109
->thenUnset()
11071110
->end()
11081111
->validate()
1109-
->always(function ($v) {
1112+
->always(function ($v) use ($nodeName) {
11101113
$isExclusive = null;
11111114
if (isset($v['type'])) {
11121115
$isExclusive = 'exclusive' === $v['type'];
@@ -1116,13 +1119,13 @@ private function addChannelsSection(ArrayNodeDefinition $handlerNode)
11161119
foreach ($v['elements'] as $element) {
11171120
if (0 === strpos($element, '!')) {
11181121
if (false === $isExclusive) {
1119-
throw new InvalidConfigurationException('Cannot combine exclusive/inclusive definitions in channels list.');
1122+
throw new InvalidConfigurationException(\sprintf('Cannot combine exclusive/inclusive definitions in %s list.', $nodeName));
11201123
}
11211124
$elements[] = substr($element, 1);
11221125
$isExclusive = true;
11231126
} else {
11241127
if (true === $isExclusive) {
1125-
throw new InvalidConfigurationException('Cannot combine exclusive/inclusive definitions in channels list');
1128+
throw new InvalidConfigurationException(\sprintf('Cannot combine exclusive/inclusive definitions in %s list', $nodeName));
11261129
}
11271130
$elements[] = $element;
11281131
$isExclusive = false;
@@ -1141,7 +1144,7 @@ private function addChannelsSection(ArrayNodeDefinition $handlerNode)
11411144
->scalarNode('type')
11421145
->validate()
11431146
->ifNotInArray(['inclusive', 'exclusive'])
1144-
->thenInvalid('The type of channels has to be inclusive or exclusive')
1147+
->thenInvalid(\sprintf('The type of %s has to be inclusive or exclusive', $nodeName))
11451148
->end()
11461149
->end()
11471150
->arrayNode('elements')

‎src/DependencyInjection/MonologExtension.php‎

Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -76,6 +76,7 @@ public function load(array $configs, ContainerBuilder $container)
7676
$handlers[$handler['priority']][] = [
7777
'id' => $this->buildHandler($container, $name, $handler),
7878
'channels' => empty($handler['channels']) ? null : $handler['channels'],
79+
'use_default_channels' => $handler['use_default_channels'],
7980
];
8081
}
8182

@@ -92,10 +93,19 @@ public function load(array $configs, ContainerBuilder $container)
9293
}
9394
}
9495

96+
$defaultChannels = $config['handler_default_channels'] ?? null;
9597
$handlersToChannels = [];
9698
foreach ($sortedHandlers as $handler) {
9799
if (!\in_array($handler['id'], $this->nestedHandlers)) {
98-
$handlersToChannels[$handler['id']] = $handler['channels'];
100+
$channels = $handler['channels'];
101+
if (null !== $defaultChannels && $handler['use_default_channels']) {
102+
if (null === $channels) {
103+
$channels = $defaultChannels;
104+
} elseif ($channels['type'] === $defaultChannels['type']) {
105+
$channels['elements'] = array_unique(array_merge($channels['elements'], $defaultChannels['elements']));
106+
}
107+
}
108+
$handlersToChannels[$handler['id']] = $channels;
99109
}
100110
}
101111
$container->setParameter('monolog.handlers_to_channels', $handlersToChannels);

‎tests/DependencyInjection/ConfigurationTest.php‎

Lines changed: 47 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -324,6 +324,31 @@ public function testWithType()
324324
$this->assertEquals('B', $config['handlers']['foo']['channels']['elements'][1]);
325325
}
326326

327+
public function testWithUseDefaultChannels()
328+
{
329+
$configs = [
330+
[
331+
'handlers' => [
332+
'foo' => [
333+
'type' => 'stream',
334+
'path' => '/foo',
335+
'use_default_channels' => true,
336+
],
337+
'bar' => [
338+
'type' => 'stream',
339+
'path' => '/bar',
340+
'use_default_channels' => false,
341+
],
342+
],
343+
],
344+
];
345+
346+
$config = $this->process($configs);
347+
348+
$this->assertTrue($config['handlers']['foo']['use_default_channels']);
349+
$this->assertFalse($config['handlers']['bar']['use_default_channels']);
350+
}
351+
327352
public function testWithFilePermission()
328353
{
329354
$configs = [
@@ -555,6 +580,28 @@ public static function processPsr3MessagesProvider(): iterable
555580
];
556581
}
557582

583+
/**
584+
* @dataProvider provideHandlerDefaultChannels
585+
*/
586+
public function testHandlerDefaultChannels($configuration, ?array $processedConfiguration)
587+
{
588+
$config = $this->process([['handler_default_channels' => $configuration]]);
589+
590+
$this->assertEquals($processedConfiguration, $config['handler_default_channels']);
591+
}
592+
593+
public static function provideHandlerDefaultChannels(): iterable
594+
{
595+
yield 'None' => [null, null];
596+
yield 'Empty array' => [[], null];
597+
yield 'As string' => ['!foo', ['type' => 'exclusive', 'elements' => ['foo']]];
598+
yield 'As array' => [['foo', 'bar'], ['type' => 'inclusive', 'elements' => ['foo', 'bar']]];
599+
yield 'With elements key' => [['elements' => ['!foo', '!bar']], ['type' => 'exclusive', 'elements' => ['foo', 'bar']]];
600+
yield 'With type key' => [['type' => 'exclusive', 'elements' => ['!foo']], ['type' => 'exclusive', 'elements' => ['foo']]];
601+
yield 'XML' => [['channel' => ['foo', 'bar']], ['type' => 'inclusive', 'elements' => ['foo', 'bar']]];
602+
yield 'XML with type' => [['type' => 'exclusive', 'channel' => ['!foo']], ['type' => 'exclusive', 'elements' => ['foo']]];
603+
}
604+
558605
/**
559606
* Processes an array of configurations and returns a compiled version.
560607
*

‎tests/DependencyInjection/FixtureMonologExtensionTestCase.php‎

Lines changed: 55 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -145,6 +145,61 @@ public function testHandlersWithChannels()
145145
);
146146
}
147147

148+
public function testHandlersWithDefaultChannels()
149+
{
150+
$container = $this->getContainer('handlers_with_default_channels');
151+
152+
$this->assertFalse($container->hasParameter('monolog.handler_default_channels'));
153+
154+
$this->assertEquals(
155+
[
156+
'monolog.handler.one' => ['type' => 'inclusive', 'elements' => ['foo']],
157+
'monolog.handler.two' => ['type' => 'exclusive', 'elements' => ['bar', 'baz']],
158+
'monolog.handler.three' => ['type' => 'exclusive', 'elements' => ['bar', 'baz', 'foo']],
159+
'monolog.handler.four' => ['type' => 'exclusive', 'elements' => ['foo', 'bar']],
160+
'monolog.handler.five' => null,
161+
'monolog.handler.six' => ['type' => 'exclusive', 'elements' => ['foo', 'bar']],
162+
],
163+
$container->getParameter('monolog.handlers_to_channels')
164+
);
165+
166+
$this->assertTrue($container->hasDefinition('monolog.logger'));
167+
$this->assertTrue($container->hasDefinition('monolog.logger.foo'));
168+
$this->assertTrue($container->hasDefinition('monolog.logger.bar'));
169+
$this->assertTrue($container->hasDefinition('monolog.logger.baz'));
170+
$this->assertTrue($container->hasDefinition('monolog.handler.one'));
171+
$this->assertTrue($container->hasDefinition('monolog.handler.two'));
172+
$this->assertTrue($container->hasDefinition('monolog.handler.three'));
173+
$this->assertTrue($container->hasDefinition('monolog.handler.four'));
174+
$this->assertTrue($container->hasDefinition('monolog.handler.five'));
175+
$this->assertTrue($container->hasDefinition('monolog.handler.six'));
176+
177+
$logger = $container->getDefinition('monolog.logger');
178+
$this->assertCount(6, $logger->getMethodCalls());
179+
$this->assertDICDefinitionMethodCallAt(5, $logger, 'pushHandler', [new Reference('monolog.handler.two')]);
180+
$this->assertDICDefinitionMethodCallAt(4, $logger, 'pushHandler', [new Reference('monolog.handler.three')]);
181+
$this->assertDICDefinitionMethodCallAt(3, $logger, 'pushHandler', [new Reference('monolog.handler.four')]);
182+
$this->assertDICDefinitionMethodCallAt(2, $logger, 'pushHandler', [new Reference('monolog.handler.five')]);
183+
$this->assertDICDefinitionMethodCallAt(1, $logger, 'pushHandler', [new Reference('monolog.handler.six')]);
184+
$this->assertDICDefinitionMethodCallAt(0, $logger, 'useMicrosecondTimestamps', ['%monolog.use_microseconds%']);
185+
186+
$logger = $container->getDefinition('monolog.logger.foo');
187+
$this->assertCount(3, $logger->getMethodCalls());
188+
$this->assertDICDefinitionMethodCallAt(2, $logger, 'pushHandler', [new Reference('monolog.handler.one')]);
189+
$this->assertDICDefinitionMethodCallAt(1, $logger, 'pushHandler', [new Reference('monolog.handler.two')]);
190+
$this->assertDICDefinitionMethodCallAt(0, $logger, 'pushHandler', [new Reference('monolog.handler.five')]);
191+
192+
$logger = $container->getDefinition('monolog.logger.bar');
193+
$this->assertCount(1, $logger->getMethodCalls());
194+
$this->assertDICDefinitionMethodCallAt(0, $logger, 'pushHandler', [new Reference('monolog.handler.five')]);
195+
196+
$logger = $container->getDefinition('monolog.logger.baz');
197+
$this->assertCount(3, $logger->getMethodCalls());
198+
$this->assertDICDefinitionMethodCallAt(2, $logger, 'pushHandler', [new Reference('monolog.handler.four')]);
199+
$this->assertDICDefinitionMethodCallAt(1, $logger, 'pushHandler', [new Reference('monolog.handler.five')]);
200+
$this->assertDICDefinitionMethodCallAt(0, $logger, 'pushHandler', [new Reference('monolog.handler.six')]);
201+
}
202+
148203
/** @group legacy */
149204
public function testSingleEmailRecipient()
150205
{

‎tests/DependencyInjection/Fixtures/xml/handlers_with_default_channels.xml‎

Lines changed: 40 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,40 @@
1+
<?xml version="1.0" ?>
2+
3+
<container xmlns="http://symfony.com/schema/dic/services"
4+
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
5+
xmlns:monolog="http://symfony.com/schema/dic/monolog"
6+
xsi:schemaLocation="http://symfony.com/schema/dic/services http://symfony.com/schema/dic/services/services-1.0.xsd
7+
http://symfony.com/schema/dic/monolog http://symfony.com/schema/dic/monolog/monolog-1.0.xsd">
8+
9+
<monolog:config>
10+
<monolog:channel>foo</monolog:channel>
11+
<monolog:channel>bar</monolog:channel>
12+
<monolog:channel>baz</monolog:channel>
13+
14+
<monolog:handler-default-channels>
15+
<monolog:channel>!foo</monolog:channel>
16+
<monolog:channel>!bar</monolog:channel>
17+
</monolog:handler-default-channels>
18+
19+
<monolog:handler name="one" type="stream" use-default-channels="true">
20+
<monolog:channels>
21+
<monolog:channel>foo</monolog:channel>
22+
</monolog:channels>
23+
</monolog:handler>
24+
<monolog:handler name="two" type="stream" use-default-channels="false">
25+
<monolog:channels>
26+
<monolog:channel>!bar</monolog:channel>
27+
<monolog:channel>!baz</monolog:channel>
28+
</monolog:channels>
29+
</monolog:handler>
30+
<monolog:handler name="three" type="stream" use-default-channels="true">
31+
<monolog:channels>
32+
<monolog:channel>!bar</monolog:channel>
33+
<monolog:channel>!baz</monolog:channel>
34+
</monolog:channels>
35+
</monolog:handler>
36+
<monolog:handler name="four" type="stream" use-default-channels="true" />
37+
<monolog:handler name="five" type="stream" use-default-channels="false" />
38+
<monolog:handler name="six" type="stream" />
39+
</monolog:config>
40+
</container>

‎tests/DependencyInjection/Fixtures/yml/handlers_with_default_channels.yml‎

Lines changed: 24 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,24 @@
1+
monolog:
2+
channels: [ 'foo', 'bar', 'baz' ]
3+
handler_default_channels: [ '!foo', '!bar' ]
4+
handlers:
5+
one:
6+
type: stream
7+
use_default_channels: true
8+
channels: foo
9+
two:
10+
type: stream
11+
use_default_channels: false
12+
channels: [ '!bar', '!baz' ]
13+
three:
14+
type: stream
15+
use_default_channels: true
16+
channels: [ '!bar', '!baz' ]
17+
four:
18+
type: stream
19+
use_default_channels: true
20+
five:
21+
type: stream
22+
use_default_channels: false
23+
six:
24+
type: stream

0 commit comments

Comments
 (0)