Remove deprecated Logger

[thekid]

The util.log.Logger class has been deprecated since 8.0.0; and is scheduled for removal in 9.0.0

Should there be a replacement for its configure() method? Should we adapt INI file layout at the same time?

[thekid]

See https://docs.python.org/2/library/logging.config.html

[handler_hand05]
class=handlers.SysLogHandler
level=ERROR
formatter=form05
args=(('localhost', handlers.SYSLOG_UDP_PORT), handlers.SysLogHandler.LOG_USER)

[johannes85]

I think thy python ini file layout is a much better approach,

[thekid]

Idea for a really short syntax

[default]
appenders[]=util.log.ConsoleAppender("err")
appenders[]=ALL:util.log.FileAppender("/var/log/server.log")
appenders[]=WARN,ERROR:util.log.SyslogUdpAppender("127.0.0.1", 514, "server")

[johannes85]

Your approach is also a good alternative since it is very compact but not that easy to read.

[thekid]

Mapping the Python example

Ini file

[default]
uses=console|syslog|files

[console]
class=util.log.ConsoleAppender
level=ALL

[files]
class=util.log.FileAppender
args="/var/log/server.log"
level=ALL

[syslog]
class=util.log.SyslogUdpAppender
args=127.0.0.1|514|server
level=WARN|ERROR

API

$c= new LogConfiguration(new Properties('log.ini'));
$cat= $c->category('default');   // Returns console, syslog and files
$cat= $c->category('syslog');    // An appender as configured by "syslog"

$all= $c->categories();  // ["default" => new LogCategory()->withAppenders(...), "syslog" => ...

$name= 'does-not-exist';
$exists= $c->provides($name);    // FALSE
$cat= $c->category($name);       // ***lang.IllegalArgumentException***

[johannes85]

👍