8000 [WIP] [Console] Add basic support for automatic console exception logging by jameshalsall · Pull Request #19382 · symfony/symfony · GitHub
[go: up one dir, main page]
Skip to content

[WIP] [Console] Add basic support for automatic console exception logging #19382

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 15 commits into from
Closed

[WIP] [Console] Add basic support for automatic console exception logging #19382

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
4284161
Add basic support for automatic console exception logging
jameshalsall Jul 18, 2016
17ef8db
Fix coding standards
jameshalsall Jul 18, 2016
3d6b22b
Simplify the log entry for console exceptions
jameshalsall Jul 21, 2016
0961f8c
Remove unnecessary exit code handling in ExceptionListener
jameshalsall Aug 9, 2016
5bd799e
Fix onKernelTerminate message logging
jameshalsall Aug 9, 2016
80c3ac3
Fix coding style
jameshalsall Aug 9, 2016
f7a7048
Fix code style
Aug 17, 2016
c21d206
Fix code style, add TODO
Aug 17, 2016
2874431
Update console error log with placeholders and context
Aug 17, 2016
bcbc8cb
Replace global usage with ArgvInput
Aug 17, 2016
da278e6
Remove backticks around command name in log message
jameshalsall Aug 17, 2016
44a0822
Add test for onKernelTerminate
jameshalsall Aug 17, 2016
5dc8b61
Update console ExceptionListener messaging, add test
jameshalsall Aug 18, 2016
b196557
Add tests for onKernelTerminate, fix log message
jameshalsall Aug 18, 2016
e6c4994
Revert change to composer.json
jameshalsall Aug 18, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix code style, add TODO
  • Loading branch information
James Halsall committed Aug 17, 2016
commit c21d2065aed7fb2064704b589d7884abc7cd2a5a
7 changes: 2 additions & 5 deletions src/Symfony/Component/Console/EventListener/ExceptionListener.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -62,11 +62,8 @@ public function onKernelTerminate(ConsoleTerminateEvent $event)
return;
}

$message = sprintf(
'Command `%s` exited with status code %d',
implode(' ', $_SERVER['argv']),
$exitCode
);
// TODO: replace $_SERVER global usage with data from Input
$message = sprintf('Command `%s` exited with status code %d', implode(' ', $_SERVER['argv']), $exitCode);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What about a command placeholder (#15753)?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍


$this->logger->error($message);
}
Expand Down
0