zoran-cuckovic / QGIS-visibility-analysis

Quantum GIS plugin for visibility analysis
GNU General Public License v3.0
72 stars 17 forks source link

Empty Try Catch Blocks #61

Closed grl-sage closed 6 months ago

grl-sage commented 2 years ago

Any reason you perform a "pass" on many of the try/catch blocks? Fortify Scans do not like those and IT is making us modify the code so we get a clean scan. The reasoning is that the code ignores an exception, which could cause the program to overlook unexpected states and conditions.

Any future updates in the works for the plugin where these may be populated by at least appending to the "error" logging variable like in other parts of the code so we don't have to maintain code just to keep the try/catch blocks from being flagged as an issue?

zoran-cuckovic commented 2 years ago

Let me get this straight : you work in the commercial sector and you'd like me to adjust the code for your needs ? You can DM me a proposal if you wish. Also, could you please provide the link to the repository, as stipulated by the GPL licence.