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

Debugging my src/main.py through GPT as part of making project showcase the strength of expert prompting & analysis #1

Open
Aureum01 opened this issue May 15, 2023 · 0 comments

Comments

@Aureum01
Copy link
Owner

According to ChatGPT 4, despite my extensively built 'main.py' and corrected sections that are logic-based on the logic of GPT 3.5 instead of mine. GPT 4 corrected 3.5 by adding:

The "check_sqli_vulnerability, is_sqli_successful, and process_successful_sqli methods are defined but not called anywhere in the code. You might want to use these methods in your SQLi testing methods (test_sqli, test_sqli_error_based, test_sqli_union_based)."

While "the SQLi detection logic in is_sqli_successful method is quite basic, checking if the response text contains "Error" or "SQL syntax". You might want to enhance this based on the specific application you're testing."

Whereas, on my end it pointed out issues such as:

"In the SQLi testing methods, you've used response.elapsed.total_seconds() > 10 to detect a successful SQLi attack. This is specific to time-based SQLi and might not be applicable for error-based and union-based SQLi.

The parse_sitemap method is defined but it's empty and not used. You should implement this method and use it to parse the sitemap and check each URL for SQLi vulnerabilities.

You might also want to handle the case where a domain is found to be vulnerable by multiple SQLi testing methods. In the current code, it will be added to self.vulnerable_domains multiple times.

The current script does not support authentication. If the site requires authentication, the tool will not work properly."

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

No branches or pull requests

1 participant