zap-extensions icon indicating copy to clipboard operation
zap-extensions copied to clipboard

spider: SAST (SonarLint) Fixes

Open kingthorin opened this issue 5 months ago • 1 comments

Overview

  • Use isEmpty vs length checks.
  • Use instanceof assignment where applicable.
  • Use text blocks vs concatenation where applicable.
  • Use computeIfAbsent vs get/put where applicable.
  • Use charset constants from JRE vs string literals.
  • Use constants from base classes vs extended classes.
  • Remove useless conditions/assignments.

Related Issues

n/a

Checklist

  • [ ] Update help
  • [x] Update changelog
  • [x] Run ./gradlew spotlessApply for code formatting
  • [ ] Write tests
  • [x] Check code coverage
  • [x] Sign-off commits
  • [x] Squash commits
  • [x] Use a descriptive title

kingthorin avatar Jun 13 '25 11:06 kingthorin

Logo Checkmarx One – Scan Summary & Detailsf49ee7fa-231e-45be-9160-633048f47566

Great job! No new security vulnerabilities introduced in this pull request


Use @Checkmarx to reach out to us for assistance.

Just send a PR comment with @Checkmarx followed by a natural language request.

Examples: @Checkmarx how are you able to help me? @Checkmarx rescan this PR

psiinon avatar Jun 13 '25 12:06 psiinon