Chore(nessusExport): clean up and refactor nessus export - #245
Chore(nessusExport): clean up and refactor nessus export#245kevjablonski wants to merge 3 commits into
Quality Gate passed
Annotations
Check failure on line 252 in server/utils/excelExport/nessusExport.ts
sonarqubecloud / SonarCloud Code Analysis
Refactor this function to reduce its Cognitive Complexity from 28 to the 15 allowed.
See more on https://sonarcloud.io/project/issues?id=mitre_tir&issues=AZ_35-_jnnsrpcWkvjI6&open=AZ_35-_jnnsrpcWkvjI6&pullRequest=245
Check warning on line 55 in db/models/nessusReportItem.ts
sonarqubecloud / SonarCloud Code Analysis
Make this public static property readonly.
See more on https://sonarcloud.io/project/issues?id=mitre_tir&issues=AZ_35-_ynnsrpcWkvjI9&open=AZ_35-_ynnsrpcWkvjI9&pullRequest=245
Check warning on line 308 in server/utils/excelExport/nessusExport.ts
sonarqubecloud / SonarCloud Code Analysis
Simplify this regular expression to reduce its runtime, as it has super-linear performance due to backtracking.
See more on https://sonarcloud.io/project/issues?id=mitre_tir&issues=AZ_35-_jnnsrpcWkvjI7&open=AZ_35-_jnnsrpcWkvjI7&pullRequest=245
Check warning on line 234 in server/utils/excelExport/nessusExport.ts
sonarqubecloud / SonarCloud Code Analysis
Prefer `.some(…)` over `.find(…)`.
See more on https://sonarcloud.io/project/issues?id=mitre_tir&issues=AZ_35-_jnnsrpcWkvjI5&open=AZ_35-_jnnsrpcWkvjI5&pullRequest=245
Check warning on line 22 in server/api/boundaries/nessus.get.ts
sonarqubecloud / SonarCloud Code Analysis
Prefer `Number.parseInt` over `parseInt`.
See more on https://sonarcloud.io/project/issues?id=mitre_tir&issues=AZ_35-8hnnsrpcWkvjIy&open=AZ_35-8hnnsrpcWkvjIy&pullRequest=245
Check failure on line 91 in server/utils/excelExport/nessusExport.ts
sonarqubecloud / SonarCloud Code Analysis
Refactor this function to reduce its Cognitive Complexity from 74 to the 15 allowed.
See more on https://sonarcloud.io/project/issues?id=mitre_tir&issues=AZ_35-_jnnsrpcWkvjI1&open=AZ_35-_jnnsrpcWkvjI1&pullRequest=245
Check warning on line 84 in server/utils/excelExport/nessusExport.ts
sonarqubecloud / SonarCloud Code Analysis
Prefer `String#replaceAll()` over `String#replace()`.
See more on https://sonarcloud.io/project/issues?id=mitre_tir&issues=AZ_35-_jnnsrpcWkvjI0&open=AZ_35-_jnnsrpcWkvjI0&pullRequest=245
Check warning on line 171 in server/utils/excelExport/nessusExport.ts
sonarqubecloud / SonarCloud Code Analysis
Expected a `for-of` loop instead of a `for` loop with this simple iteration.
See more on https://sonarcloud.io/project/issues?id=mitre_tir&issues=AZ_35-_jnnsrpcWkvjI3&open=AZ_35-_jnnsrpcWkvjI3&pullRequest=245
Check warning on line 172 in server/utils/excelExport/nessusExport.ts
sonarqubecloud / SonarCloud Code Analysis
Expected a `for-of` loop instead of a `for` loop with this simple iteration.
See more on https://sonarcloud.io/project/issues?id=mitre_tir&issues=AZ_35-_jnnsrpcWkvjI2&open=AZ_35-_jnnsrpcWkvjI2&pullRequest=245
Check failure on line 365 in server/utils/excelExport/nessusExport.ts
sonarqubecloud / SonarCloud Code Analysis
Refactor this function to reduce its Cognitive Complexity from 16 to the 15 allowed.
See more on https://sonarcloud.io/project/issues?id=mitre_tir&issues=AZ_35-_jnnsrpcWkvjI8&open=AZ_35-_jnnsrpcWkvjI8&pullRequest=245
Check warning on line 24 in server/api/boundaries/nessus.get.ts
sonarqubecloud / SonarCloud Code Analysis
Prefer `Number.isNaN` over `isNaN`.
See more on https://sonarcloud.io/project/issues?id=mitre_tir&issues=AZ_35-8hnnsrpcWkvjIz&open=AZ_35-8hnnsrpcWkvjIz&pullRequest=245
Check warning on line 170 in server/utils/excelExport/nessusExport.ts
sonarqubecloud / SonarCloud Code Analysis
Expected a `for-of` loop instead of a `for` loop with this simple iteration.
See more on https://sonarcloud.io/project/issues?id=mitre_tir&issues=AZ_35-_jnnsrpcWkvjI4&open=AZ_35-_jnnsrpcWkvjI4&pullRequest=245