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

Config file verification happens too late during startup and does not always exit the program #857

Closed
mountaindude opened this issue Aug 19, 2024 · 0 comments · Fixed by #848 · May be fixed by #936
Closed

Config file verification happens too late during startup and does not always exit the program #857

mountaindude opened this issue Aug 19, 2024 · 0 comments · Fixed by #848 · May be fixed by #936
Labels
bug delivered This feature has been included in Butler SOS

Comments

@mountaindude
Copy link
Collaborator

What version of Butler SOS are you using?

10.1.1

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

20

What command did you use to start Butler SOS?

node ./src/butler-sos.js --loglevel info --configfile ./src/config/production.yaml

What operating system are you using?

Linux

What CPU architecture are you using?

x86_64

What Qlik Sense versions are you using?

2024 May

Describe the Bug

  1. Config file verification should happen as early during startup as possible
  2. In some cases a failed verification does not exit Butler SOS (which should be the case in that scenario).
  3. For example, when some array's (in the config file) parent does not exist. In that case there is not a perfect feedback message from Butler SOS about what is missing/incorrect, and Butler SOS keeps running.

Expected Behavior

  1. Move verification to beginning of app startup
  2. Hard to give better error message (possible but takes lots of work, no clear way of solving it), but Butler SOS should at least ALWAYS exit when incorrect config file format is detected.

To Reproduce

No response

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug delivered This feature has been included in Butler SOS
Projects
None yet
1 participant