Codeql

[ldoktor]

Enable LGTM directly on GH and fix the issues we want to address.

closes: #109

[lgtm-com[bot]]

This pull request introduces 1 alert and fixes 3 when merging 76181f4f5459357082fc78136c2001685d0bcaa5 into f7ceac6a2eb377c73ed5e6e9064894b313a4e282 - view on LGTM.com

new alerts:

• 1 for Clear-text logging of sensitive information

fixed alerts:

• 1 for Unused local variable
• 1 for Variable defined multiple times

• 1 for Insecure temporary file

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine :gear: that powers LGTM.com. For more information, please check out our post on the GitHub blog.

[ldoktor]

Thank you, Lucas, for review, I added extra commit to treat null/empty output_params, which was polluting my logs.

[lgtm-com[bot]]

This pull request introduces 1 alert and fixes 3 when merging 055fbe34372db811bec6448c97462d30a93b8753 into f7ceac6a2eb377c73ed5e6e9064894b313a4e282 - view on LGTM.com

new alerts:

• 1 for Clear-text logging of sensitive information

fixed alerts:

• 1 for Unused local variable
• 1 for Variable defined multiple times

• 1 for Insecure temporary file

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine :gear: that powers LGTM.com. For more information, please check out our post on the GitHub blog.

[ldoktor]

Changes:

Split the lock/open warning messages

[lgtm-com[bot]]

This pull request introduces 1 alert and fixes 3 when merging fd03013802e0d1b2b2cac2a815617106b836af4a into f7ceac6a2eb377c73ed5e6e9064894b313a4e282 - view on LGTM.com

new alerts:

• 1 for Clear-text logging of sensitive information

fixed alerts:

• 1 for Unused local variable
• 1 for Variable defined multiple times

• 1 for Insecure temporary file

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine :gear: that powers LGTM.com. For more information, please check out our post on the GitHub blog.