Skip to content

Conversation

@ganesh-dagadi
Copy link
Contributor

@ganesh-dagadi ganesh-dagadi commented Jan 28, 2024

Overview

Used Comparable response instead of Response matcher in SQL injection plugin. Part of zaproxy/zaproxy#7116

Related Issues

Issue Link

Checklist

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

For more details, please refer to the developer rules and guidelines.

@github-actions
Copy link

github-actions bot commented Jan 28, 2024

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@thc202 thc202 changed the title SQLiPlugin: Use Comparable Response instead of Response Matcher sqliplugin: Use Comparable Response instead of Response Matcher Jan 28, 2024
@ganesh-dagadi
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA

@ganesh-dagadi

This comment was marked as outdated.

zapbot added a commit to zaproxy/cla that referenced this pull request Jan 28, 2024
@ganesh-dagadi
Copy link
Contributor Author

I have encountered an Issue with the SQLi plugin. Please have a look at this
zaproxy/zaproxy#7116 (comment)

@kingthorin
Copy link
Member

You can change the logic if necessary as long as the end result/behaviour is the same.

@ganesh-dagadi
Copy link
Contributor Author

Thank you for the suggestion. I will test the changes thoroughly and ensure the behavior remains consistent. I will update the results as soon as possible.

@psiinon
Copy link
Member

psiinon commented Jan 19, 2025

Logo
Checkmarx One – Scan Summary & Details73a98c03-ae54-477f-9bd9-383a2dbd11d1

Great job, no security vulnerabilities found in this Pull Request

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants