diff --git a/src/Monolog/Processor/WebProcessor.php b/src/Monolog/Processor/WebProcessor.php index 7b95b0e1..6c32b2d1 100644 --- a/src/Monolog/Processor/WebProcessor.php +++ b/src/Monolog/Processor/WebProcessor.php @@ -52,6 +52,10 @@ class WebProcessor implements ProcessorInterface throw new \UnexpectedValueException('$serverData must be an array or object implementing ArrayAccess.'); } + if (isset($this->serverData['UNIQUE_ID'])) { + $this->extraFields['unique_id'] = 'UNIQUE_ID'; + } + if (null !== $extraFields) { if (isset($extraFields[0])) { foreach (array_keys($this->extraFields) as $fieldName) { @@ -91,10 +95,6 @@ class WebProcessor implements ProcessorInterface $extra[$extraName] = $this->serverData[$serverName] ?? null; } - if (isset($this->serverData['UNIQUE_ID'])) { - $extra['unique_id'] = $this->serverData['UNIQUE_ID']; - } - return $extra; } } diff --git a/tests/Monolog/Processor/WebProcessorTest.php b/tests/Monolog/Processor/WebProcessorTest.php index 463dfd98..3f994893 100644 --- a/tests/Monolog/Processor/WebProcessorTest.php +++ b/tests/Monolog/Processor/WebProcessorTest.php @@ -88,6 +88,19 @@ class WebProcessorTest extends TestCase $this->assertSame(['url' => 'A', 'http_method' => 'C'], $record['extra']); } + public function testProcessorAddsOnlyRequestedExtraFieldsIncludingOptionalFields() + { + $server = array( + 'REQUEST_URI' => 'A', + 'UNIQUE_ID' => 'X', + ); + + $processor = new WebProcessor($server, array('url')); + $record = $processor($this->getRecord()); + + $this->assertSame(array('url' => 'A'), $record['extra']); + } + public function testProcessorConfiguringOfExtraFields() { $server = [