-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Contrast adr solution #11864
base: master
Are you sure you want to change the base?
Contrast adr solution #11864
Conversation
@microsoft-github-policy-service agree company="Contrast Security" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typo: ContrastADR_Command_Injestion.yaml
-> ContrastADR_Command_Injection.yaml
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typo: Contrast_ADR_SQL_ingestion.yaml
-> Contrast_ADR_SQL_Injection.yaml
id: 223db5c1-1bf8-47d8-8806-bed401b356a4 | ||
name: SQL Injection | ||
description: | | ||
'Critical Sevierty Level.' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'Critical Sevierty Level.' | |
'Critical Severity Level.' |
id: 223db5c1-1bf8-47d8-8806-bed401b356a4 | ||
name: SQL Injection | ||
description: | | ||
'High Sevierty Level.' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'High Sevierty Level.' | |
'High Severity Level.' |
id: 223db5c1-1bf8-47d8-8806-bed401b356a4 | ||
name: SQL Injection | ||
description: | | ||
'Low Sevierty Level.' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'Low Sevierty Level.' | |
'Low Severity Level.' |
id: 223db5c1-1bf8-47d8-8806-bed401b356a4 | ||
name: SQL Injection | ||
description: | | ||
'Medium Sevierty Level.' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'Medium Sevierty Level.' | |
'Medium Severity Level.' |
Hello @PavanContrast, By any chance did you raised a duplicate PR #11863 ? |
"""A decorator to retry a function/method upon failure.""" | ||
|
||
def decorator(func): | ||
def wrapper(*args, **kwargs): |
Check notice
Code scanning / CodeQL
Explicit returns mixed with implicit (fall through) returns
except Exception as e: | ||
# Fallback to using webhook data | ||
logging.error(f"Enrichment API call failed: {e}") | ||
response_data = self.req_body |
Check notice
Code scanning / CodeQL
Unused local variable
Yes it was a duplicate PR, closed PR #11863 |
For all the workbooks please add meta data to workbooksmetadata file. Also please add images of workbook with black and white backgrounds. |
- connectorId: ContrastADR | ||
dataTypes: | ||
- ContrastADR_CL | ||
queryFrequency: 5M |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Time identifier needs to be corrected to 5m in all the analytic rules
Please provide me branch access so that I can commit some required changes |
…nel-New into Contrast-ADR-Solution
af911dc
to
ff4b010
Compare
|
Change(s):
Reason for Change(s):
Version Updated:
Testing Completed:
Checked that the validations are passing and have addressed any issues that are present: