worker: allow to set logFile as /dev/null #993
Conversation
| static::$logFile ??= sprintf('%s/workerman.log', dirname(__DIR__, 2)); | ||
|
|
||
| if (!is_file(static::$logFile)) { | ||
| if (!is_file(static::$logFile) && static::$logFile !== '/dev/null') { |
There was a problem hiding this comment.
Should be better to replace is_file with file_exists function. file_exists can handle /dev/null and other special files.
There was a problem hiding this comment.
@luzrain Thanx!
I didn't want to change the semantics of the code, I only allowed /dev/null
if you insist, please make your own fix
There was a problem hiding this comment.
We need to support more options, not only /dev/null like stderr, syslog, database, net, ...
I think that we need to work more with the log with error and access later:
- configure log which data to show
- have more than 1 log
- error log only show by severity (warn, info, ...)
- ...
There was a problem hiding this comment.
maybe we introduce $noLog option as first step?
There was a problem hiding this comment.
We need to create a Log Class, to easily add more options or that a user extend the class in the way he need.
There was a problem hiding this comment.
This is very difficult task for me in general
There was a problem hiding this comment.
I know that Walkor want Workerman minimalist, and I think like him.
But a minimal Log Class, that any user could change to his needs will be a good feature.
There was a problem hiding this comment.
Actually we can do the same than nodejs.
node my_app.js > my_app_log.log 2> my_app_err.log
Workerman only log in daemon mode, in debug mode use STDOUT
php start.php start > /dev/null 2> my_app_err.log
Theorically it will work.
4 participants
This small code allows to use /dev/null as logFile