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

Fixes an issue with certain reports #974

Merged
merged 1 commit into from
Mar 23, 2019
Merged

Conversation

adracea
Copy link
Contributor

@adracea adracea commented Mar 23, 2019

Uploading a burp suite report that contained certain characters ended up with throwing errors due to the str(self.description) which would end up with a partial report upload .
For example the "Interesting input handling: Magic value: null" issue generated by the Burp Pro Plugin Backslash Powered Scanner ended up not uploading due to the fact that it contained a multi-line table .

Please submit your pull requests to the 'dev' branch.

When submitting a pull request, please make sure you have completed the following checklist:

  • Your code is flake8 compliant (DefectDojo's code isn't currently flake8 compliant, but we're trying to correct that.)
  • If this is a new feature and not a bug fix, you've included the proper documentation in the ReadTheDocs documentation folder. https://github.com/DefectDojo/Documentation/tree/master/docs or provide feature documentation in the PR.
  • Add applicable tests to the unit tests.

Uploading a burp suite report that contained certain characters ended up with throwing errors due to the `str(self.description)` which would end up with a partial report upload .
For example the "Interesting input handling: Magic value: null" issue generated by the Burp Pro Plugin Backslash Powered Scanner ended up not uploading due to the fact that it contained a multi-line table .
@aaronweaver aaronweaver merged commit d1309bc into DefectDojo:dev Mar 23, 2019
This was referenced Apr 3, 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.

2 participants