Skip to content

Commit

Permalink
bug #52994 [MonologBridge] Fix context data and display extra data (l…
Browse files Browse the repository at this point in the history
…ouismariegaborit)

This PR was merged into the 6.3 branch.

Discussion
----------

[MonologBridge] Fix context data and display extra data

| Q             | A
| ------------- | ---
| Branch?       | 6.3
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Issues        | ->
| License       | MIT

Fix the compatibility with Monolog 3. In the PR #52222 we had the compatibility with Monolog 3 but context parameter is incorrect and extra parameter is missing.

No test can be added because testing this command is not possible at this time. Another PR (#53518) was opened to add test on this command.

Commits
-------

8687ae0 Fix context data and display extra data
  • Loading branch information
nicolas-grekas committed Jan 29, 2024
2 parents 9e92ab3 + 8687ae0 commit 2fae7d5
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion src/Symfony/Bridge/Monolog/Command/ServerLogCommand.php
Expand Up @@ -164,7 +164,8 @@ private function displayLog(OutputInterface $output, int $clientId, array $recor
$record['channel'],
Level::fromValue($record['level']),
$record['message'],
$record['context']->getContext(),
$record['context']->getValue(true),
$record['extra']->getValue(true),
);
}

Expand Down

0 comments on commit 2fae7d5

Please sign in to comment.