Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

No console logging unless --loglevel parameter is specified when starting Butler SOS #925

Closed
mountaindude opened this issue Oct 21, 2024 · 3 comments · Fixed by #910 · May be fixed by #936
Closed

No console logging unless --loglevel parameter is specified when starting Butler SOS #925

mountaindude opened this issue Oct 21, 2024 · 3 comments · Fixed by #910 · May be fixed by #936
Labels

Comments

@mountaindude
Copy link
Collaborator

What version of Butler SOS are you using?

11.0.1

What version of Node.js are you using? Not applicable if you use the standalone version of Butler SOS.

18.20.3

What command did you use to start Butler SOS?

/Users/goran/.nvm/versions/node/v18.20.3/bin/node ./butler-sos.js --configfile ./config/production.yaml

What operating system are you using?

macOs

What CPU architecture are you using?

x86_64

What Qlik Sense versions are you using?

2024-May IR

Describe the Bug

When starting Butler SOS with /Users/goran/.nvm/versions/node/v18.20.3/bin/node ./butler-sos.js --configfile ./config/production.yaml the tool starts, but there is no log output what so ever.

Starting with /Users/goran/.nvm/versions/node/v18.20.3/bin/node ./butler-sos.js --configfile ./config/production.yaml --loglevel info shows the expected log output.

This is a regression error, prior to 11.0 this was not a problem.

Expected Behavior

No response

To Reproduce

No response

@mountaindude
Copy link
Collaborator Author

This bug is caused by setting logLevel to INFO rather than info in the config file.

Remove conversion to lower case in the config file schema, to throw a schema verification error if the log level is not lowercase.

@mountaindude
Copy link
Collaborator Author

Also, the config file schema is verified rather late in the startup process.
Change so it is checked as quickly as possible after the path of the config file has been determined.

@mountaindude
Copy link
Collaborator Author

Update template config with info that logLevel setting is case sensitive, should be lower case.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
1 participant