Closed f-laurens closed 1 year ago
This pull request introduces 6 alerts when merging dc65080abe9787d3adb349970c1d26934c7f0044 into e46ed3c06760591ad1656d3d6dee6b8286c1f19a - view on LGTM.com
new alerts:
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. It looks like GitHub code scanning with CodeQL is already set up for this repo, so no further action is needed :rocket:. For more information, please check out our post on the GitHub blog.
Another slight issue (not handled here) is with the module/service names, which should rather have been nabradiod and nabwebhookd, for consistency with existing services. But this would be tricky to change now, since users may have already upgraded to/installed v1.0.2.
Fix and localize application names. Cleanup logging and exception handling.