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

fix sqlmap bug #89

Merged
merged 248 commits into from
Apr 5, 2019
Merged

fix sqlmap bug #89

merged 248 commits into from
Apr 5, 2019

Conversation

Anemone95
Copy link
Contributor

When content type is 'application/json', sqlmap can't find any injections. So we should change the body type from dict to jsonstring. See modules/sqli.py for more details.

@sagarpo
Copy link
Collaborator

sagarpo commented Mar 28, 2019

@Anemone95 Thanks for the pull request. I will check this soon.

Copy link
Collaborator

@sagarpo sagarpo left a comment

Choose a reason for hiding this comment

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

Can you delete .gitignore, deployment.xml, vcs.xml and scan.log?

@Anemone95 Anemone95 force-pushed the master branch 3 times, most recently from 12cc8ac to 023d875 Compare April 5, 2019 09:54
@sagarpo sagarpo changed the base branch from master to dev April 5, 2019 10:58
@sagarpo
Copy link
Collaborator

sagarpo commented Apr 5, 2019

@Anemone95: I have merged it to dev branch. I will re-check it and merge it to master. Thanks for the fix.

@sagarpo sagarpo merged commit e1cca6f into flipkart-incubator:dev Apr 5, 2019
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.

9 participants