Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

[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 onKernelTerminate message logging
  • Loading branch information
@jameshalsall
jameshalsall authored and James Halsall committed Aug 17, 2016
commit 5bd799ee44362f3843f0c4f1366f37425c94d80d
10 changes: 8 additions & 2 deletions 10 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,9 +62,15 @@ public function onKernelTerminate(ConsoleTerminateEvent $event)
return;
}

$message = sprintf('Command `%s` exited with status code %d');
$message = sprintf(
'Command `%s` exited with status code %d',
join(' ', $_SERVER['argv']),
$exitCode
);

$this->logger->warning($message);
$this->logger->error(
Copy link
Member

Choose a reason for hiding this comment

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

We prefer single line here, same above. And in fact I think the intermediate $message is not required.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

👍 my bad

$message
);
}

public static function getSubscribedEvents()
Expand Down
Morty Proxy This is a proxified and sanitized view of the page, visit original site.