working-plusplus icon indicating copy to clipboard operation
working-plusplus copied to clipboard

Eliminate potential SQL injection from database queries

Open Alex-Vol-SV opened this issue 5 years ago • 1 comments

The TODO markers indicating the possibility of SQL injection issues were used to guide this implementation. Fixed by applying parameterized queries.

Found a unitest issue that was masked by the use of concatenation in SQL and fixed the unit tests to match the runtime code execution.

Alex-Vol-SV avatar Dec 23 '19 07:12 Alex-Vol-SV

Pull Request Test Coverage Report for Build 128

  • 4 of 4 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.3%) to 72.318%

Totals Coverage Status
Change from base Build 89: 0.3%
Covered Lines: 258
Relevant Lines: 333

💛 - Coveralls

coveralls avatar Dec 23 '19 21:12 coveralls