Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

minor updates to the SQL Injection page #986

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

zbraiterman
Copy link
Contributor

No description provided.

@@ -43,7 +41,7 @@ See the [OWASP Code Review Guide](https://owasp.org/www-project-code-review-guid

### How to Test for SQL Injection Vulnerabilities

See the [OWASP Testing Guide](https://owasp.org/www-project-web-security-testing-guide/) for information on testing for SQL Injection vulnerabilities.
See the [OWASP Testing Guide](https://github.com/OWASP/wstg/blob/master/document/4-Web_Application_Security_Testing/07-Input_Validation_Testing/05-Testing_for_SQL_Injection.md) for information on testing for SQL Injection vulnerabilities.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This shouldn't be changed. It could link to the v4.2 content published on the site but it's still probably best to just leave it as it was that way users can easily check latest, stable, or versioned WSTG content.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That makes sense, thank you. Sorry about that. By the way, I'm glad to see that you've added a section on threat modeling.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants