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

Added SQL injection detection folder #793

Merged
merged 2 commits into from
Jul 2, 2024

Conversation

Chhaya-g
Copy link
Contributor

@Chhaya-g Chhaya-g commented Jun 7, 2024

Aim: The aim of the problem statement is to develop a CNN model for SQL injection detection. I have used a huge dataset from Kaggle(68,000 rows) consisting of user input along with a label. I have trained a simple CNN model as well as a 2D CNN model to solve the problem.
@Niketkumardheeryan, To solve the problem, I added a SQL injection detection Folder with seven files readme.md, cnn.md, requirements.txt, image.png, mergecsv.csv, SQLID.ipynb, SQLinjectiondetection2DCNN.ipynb.
readme.md, image.png and cnn.md have all the necessary description about the problem and its various aspects and the underlying technique used to solve the problem.
requirements.txt contains all the requirements to run this project on a local machine.
SQLID.ipynb and SQLinjectiondetection2DCNN.ipynb contains the main model used for the problem along with all the data preprocessing and feature extraction.
Here I have used both CNN and 2D CNN for solving the problem. Kindly, look into it @invigorzz313

closes: #776

@Chhaya-g
Copy link
Contributor Author

@Niketkumardheeryan @invigorzz313 kindly check my pull request which I had opened last week in response to getting an issue assigned to me under GSSOC'24.

@Niketkumardheeryan
Copy link
Owner

@Chhaya-g add more cells in your code , for better readability and add more comments as well . Use proper heading in readme.md file with results of your model.

@Chhaya-g
Copy link
Contributor Author

@Chhaya-g add more cells in your code , for better readability and add more comments as well . Use proper heading in readme.md file with results of your model.

okay, I will get back to you asap.

@Chhaya-g
Copy link
Contributor Author

@Niketkumardheeryan I have added more comments, improved the readability of the code as well as of the markdown files(readme.md & cnn.md) and also mentioned the results of the models in the readme.md file.
Also SQLinjectiondetection2DCNN.ipynb is the main jupyter notebook with all the steps(data preprocessing and feature engineering) with comments and the other jupyter notebook just has few more main models, as all the prior steps are just the same.
Kindly, check my commit and merge it under GSSOC'24. Also please assign a necessary level label (level-2 or level-3).

@Niketkumardheeryan Niketkumardheeryan added the level2 level2 label Jul 2, 2024
@Niketkumardheeryan Niketkumardheeryan merged commit ceb1701 into Niketkumardheeryan:master Jul 2, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

SQL injection detection using CNN
2 participants