Lucee icon indicating copy to clipboard operation
Lucee copied to clipboard

Published 20 hours ago • verified publisher icon lucee

Improve extension logging 6.0 LDEV-3032

Open zspitzer opened this issue 4 years ago • 4 comments

  • tweak language, update is now installing
  • changed default deploy log level to INFO
  • changed all the verbose install entries to DEBUG
  • log when an extension manifest entry install fails due to any problem
  • log individual file installs under "real ext name" rather than generically as "extension"

https://luceeserver.atlassian.net/browse/LDEV-3032

zspitzer avatar Apr 04 '21 20:04 zspitzer

https://github.com/lucee/Lucee/pull/1190

zspitzer avatar Apr 05 '21 15:04 zspitzer

we should NOT change the default log level, unless there is really a good reason for it

michaeloffner avatar Apr 26 '21 15:04 michaeloffner

url,path,url is not necessary to get an extension, id and version can be sufficient. Actually id on it's own should be sufficient, when you define no version, it simply should pick the latest version.

michaeloffner avatar Apr 26 '21 15:04 michaeloffner

my rationale for changing the log level to info was to log out a single log entry for all installs / upgrades,

as I moved all the detailed logging to DEBUG level. logging out upgrades / installs as warnings or errors doesn't quite feel right

I'll change it back to ERROR then

zspitzer avatar Apr 26 '21 15:04 zspitzer

@cfmitrah the underlaying ticket is already set to resolved. Does this PR still address an existing issue or is it already obsolete? If it still is an issue, please create a new ticket for the issue and a new PR.

michaeloffner avatar Feb 16 '24 11:02 michaeloffner