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

🐛 Handle when Burp Rest API Json file contains binary #9131

Merged
merged 2 commits into from Dec 22, 2023

Conversation

manuel-sommer
Copy link
Contributor

@manuel-sommer manuel-sommer commented Dec 7, 2023

see issue description #9128

Copy link

dryrunsecurity bot commented Dec 7, 2023

Contextual Security Analysis

As DryRun Security performs checks, we’ll summarize them here. You can always dive into the results in the section below for checks.

Status DryRun Security Check
AI-powered Sensitive Function Check
Configured Sensitive Files Check
AI-powered Sensitive Files Check

Chat with your AI-powered Security Buddy by typing /dryrunsec: (or /drs:) followed by your question. Example: /dryrunsec: From a security perspective, what are some sensitive files in an Express application?

Install and configure more repositories at DryRun Security

@manuel-sommer
Copy link
Contributor Author

@Maffooch, @dsever and and @madchap could you take a look and merge?

Copy link
Contributor

@mtesauro mtesauro left a comment

Choose a reason for hiding this comment

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

Approved

@manuel-sommer
Copy link
Contributor Author

@dsever and @Maffooch reminder

@manuel-sommer
Copy link
Contributor Author

@grendel513 could you review please?

@Maffooch Maffooch changed the title 🐛 fix for issue #9128 🐛 Handle when Burp Rest API Json file contains binary Dec 22, 2023
@Maffooch Maffooch merged commit 9aac429 into DefectDojo:dev Dec 22, 2023
120 checks passed
@manuel-sommer manuel-sommer deleted the fix_burpapiparser branch December 29, 2023 21:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants