-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Add Simple Metrics API endpoint #12533
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
Add Simple Metrics API endpoint
405b676
Add Simple Metrics API endpoint #12533
405b676
DryRun Security
Details
General Security Analyzer Findings: 5 detected
⚠️ Metrics Information Disclosure dojo/api_v2/serializers.py (click for details)
| Type | Metrics Information Disclosure | ||||||||||||||||||||||||||||||||||||||||||||
| Description | The new SimpleMetricsSerializer exposes detailed security metrics, including finding counts by severity and status. If not properly secured, this could provide attackers with insights into the application's security posture and internal metrics. | ||||||||||||||||||||||||||||||||||||||||||||
| Filename | dojo/api_v2/serializers.py | ||||||||||||||||||||||||||||||||||||||||||||
| CodeLink | django-DefectDojo/dojo/api_v2/serializers.py Lines 3103 to 3124 in 405b676
|
⚠️ Logout IP Logging dojo/utils.py (click for details)
| Type | Logout IP Logging | ||||||||||||||||||||
| Description | The patch introduces logging of IP addresses for both authenticated and anonymous logout attempts. While logging can be useful for auditing, logging IP addresses for anonymous users could potentially be used for tracking or reconnaissance if not carefully managed. | ||||||||||||||||||||
| Filename | dojo/utils.py | ||||||||||||||||||||
| CodeLink | django-DefectDojo/dojo/utils.py Lines 2397 to 2406 in 405b676
|
⚠️ Product Type Enumeration dojo/api_v2/views.py (click for details)
| Type | Product Type Enumeration | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Description | The new metrics API endpoint allows filtering by product type ID, which could potentially enable an attacker to enumerate valid product type IDs through careful probing of the API responses. While the implementation includes authorization checks, the distinct error messages and response handling could leak information about the existence of product types. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Filename | dojo/api_v2/views.py | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| CodeLink | django-DefectDojo/dojo/api_v2/views.py Lines 3174 to 3296 in 405b676
|
⚠️ Test Credential Linting Disabled ruff.toml (click for details)
| Type | Test Credential Linting Disabled | ||||||||||||||
| Description | Disabling the S106 linting rule for test files removes warnings about hardcoded passwords, which could lead to accidental exposure of credentials if test files are mishandled or the repository is compromised. | ||||||||||||||
| Filename | ruff.toml | ||||||||||||||
| CodeLink | Lines 112 to 118 in 405b676
|
⚠️ Verbose Error Message Disclosure docs/content/en/api/metrics-endpoint.md (click for details)
| Type | Verbose Error Message Disclosure | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Description | The API documentation reveals detailed error messages that provide insights into the backend's validation logic. While informative for developers, these messages could assist an attacker in understanding the application's input validation mechanisms and potential attack surfaces. | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Filename | docs/content/en/api/metrics-endpoint.md | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| CodeLink | django-DefectDojo/docs/content/en/api/metrics-endpoint.md Lines 1 to 169 in 405b676
|