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 issue #8096 #9538

Closed

Conversation

manuel-sommer
Copy link
Contributor

see issue #8096

Copy link

dryrunsecurity bot commented Feb 13, 2024

Contextual Security Analysis

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

Status DryRun Security Check
Sensitive Functions Analyzer
Configured Sensitive Files Analyzer
Sensitive Files Analyzer

Chat with your AI-powered Security Buddy by typing @dryrunsecurity followed by your question into a comment.
Example: @dryrunsecurity What are common security issues with web application cookies?

Install and configure more repositories at DryRun Security

@manuel-sommer
Copy link
Contributor Author

Add 1 missing arguments; 'remove_finding_from_risk_acceptance' expects 2 positional arguments

--> I need help here, where can I get "risk_acceptance" argument from?

@manuel-sommer
Copy link
Contributor Author

Could you help here @Maffooch ?

@Maffooch
Copy link
Contributor

@manuel-sommer you need to pass the risk acceptance the the finding should be removed from. The ra_helper field in that instance is just an imported file

def remove_finding_from_risk_acceptance(risk_acceptance, finding):

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.

2 participants