-
Notifications
You must be signed in to change notification settings - Fork 0
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
PR Test 2 #2
base: main
Are you sure you want to change the base?
PR Test 2 #2
Conversation
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.
Introduced vulnerabilities outside the changed code
Vulnerability C on utils.js, line 20.
@@ -4,7 +4,7 @@ | |||
|
|||
// mongoose setup | |||
require('./mongoose-db'); | |||
require('./typeorm-db') | |||
require('./typeorm-db'); |
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.
Vulnerability B (✅ Resolved on commit bf4b849)
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.
Reply
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.
Introduced vulnerabilities outside the changed code
app.js
(1 of 2 Resolved)
Vulnerability A (✅ Resolved on commit bf4b849)
Line 35 in bf4b849
app.engine('dust', cons.dust); |
Vulnerability D
Line 89 in bf4b849
console.log('Express server listening on port ' + app.get('port')); |
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.
Vulnerability D on line 42
@@ -71,6 +71,8 @@ app.use('/users', routesUsers) | |||
// Static | |||
app.use(st({ path: './public', url: '/public' })); | |||
|
|||
console.log('test'); |
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.
Vulnerability B
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.
Vulnerability F on line 69
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
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.
Vulnerability C on utils.js, line 20.
Vulnerability C on utils.js, line 99. |
Vulnerability C on utils.js, line 20. |
No description provided.