2 possible minor errors (and PR)
Antero Mejr 13 Mar 2024 17:15 UTC
In the send-log specification, the following example is provided:
(send-log 'INFO (string-append "User " username " logged in")
'USERNAME username 'REMOTE_IP remote-ip)
However when I try that with the sample implementation's send-log, an
error is raised because 'INFO should be an integer, not a symbol. I
think the correct example would be:
(send-log INFO (string-append "User " username " logged in")
'USERNAME username 'REMOTE_IP remote-ip)
Also, the error message in the sample implementation when
current-log-callback is parameterized incorrectly references
"current-log-hook", which does not exist. Presumably the parameter name
was changed but the error message was not.
Pull request with these small fixes is here:
https://github.com/scheme-requests-for-implementation/srfi-215/pull/3