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 the URI encoded injection code feature #28

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

DeweshSingh
Copy link

I have modified the ga_main.py and the gan_main.py files.
Now these files not only test the normal injection code during evaluation but also the encoded version of these injection codes for detecting if the script is running. The csv files generated after running the ga_main.py and gan_main.py now contain the injection code along with their encoded version as well.

The most common XSS attack prevention technique is the regex string matching. This encoded injection code has a better chance of causing an XSS attack by not getting detected by normal string matching techniques. Thus this feature has an advantage over the normal injection codes.

@13o-bbr-bbq
Copy link
Owner

Great thanks!!
But, i cannot confirm you modified codes because your codes are included extra blanks.
Could you modify codes using PEP8 that Python's Coding Standards?

@DeweshSingh
Copy link
Author

Looks like the problem was with the indentation. I generally use tabs instead of spaces. I have fixed it now. Kindly have a look at it.

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