It's only when somthing goes wrong that you tend to start looking for the logs.. so logging should be set at a sensible level by default. For instance, the fines.pl script supports logging but it is disabled by default. I believe this is wrong, logs should be enabled, directed to /var/log/koha/instance whatever and logrotate used to manage them.
Hm, I think I disagree and the logging had been turned off by intention. I think we shouldn't put files with patron sensitive information on the server by default, when the library might not be aware of it. It should be opt in... not opt out.
Hm, also there is logging on the database side if you have FinesLog activated.
Logging should be set up like most things do it: errors, warnings, info, debug levels. My suggestion is that default, errors (and warnings?) are logged, however are checked to ensure that they won't divulge sensitive information, but is able to give you a heads-up to say that something might need checking out. Also, they should use something like Log::Any or Log4Perl to do it properly, none of this spamming syserr/sysout like some scripts do.
Hi Robin, makes sense to me.
I wonder if this should be rephrased and what would be a next step here?