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.

Sign up for GitHub

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

Jump to bottom

Update README.md #82

Merged
merged 1 commit into from
Nov 29, 2021
Merged

Update README.md #82

merged 1 commit into from
Nov 29, 2021

Conversation

rudy-marquez
Copy link
Owner

No description provided.

@rudy-marquez
Copy link
Owner Author

Scan submitted to Checkmarx

@rudy-marquez
Copy link
Owner Author

Logo
Checkmarx SAST - Scan Summary & Details

Cx-SAST Summary

Total of 17 vulnerabilities
High 13 High
Medium 4 Medium
Low 0 Low
Info 0 Info

Checkmarx Scan Summary

Severity Count
High 13
Medium 4
Low 0
Informational 0

Violation Summary

High 13 High

View more details on Checkmarx UI

Cx-SAST Details

Lines Severity Category File Link
45 74 High SQL_Injection core/authHandler.js Checkmarx
66 68 High SQL_Injection core/passport.js Checkmarx
10 87 109 147 High SQL_Injection core/appHandler.js Checkmarx
3 High Insecure_Storage_of_Sensitive_Data config/db.js Checkmarx
24 High Insecure_Storage_of_Sensitive_Data server.js Checkmarx
67 High Insecure_Storage_of_Sensitive_Data core/passport.js Checkmarx
39 197 High Code_Injection core/appHandler.js Checkmarx

@rudy-marquez rudy-marquez merged commit 9d396e0 into master Nov 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@rudy-marquez