-
Notifications
You must be signed in to change notification settings - Fork 4
Chore/improve-db-connectivity #142
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
Conversation
…ervice with better architecture
* @returns {Promise<Object>} Cleanup results | ||
*/ | ||
async manualCleanupOldDrafts(daysOld?: number) { | ||
this.logger.log( |
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.
Mend Code Security Check
New finding (4 of 5)
The Mend Code Security Check of your branch failed because of a Log Forging finding in this line.
Severity | Vulnerability Type | CWE | File | Data Flows | Detected | ||||||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Log Forging | 1 | 2025-10-06 06:28pm | |||||||||||||||||||||||||||
|
* | |
* @param {number} daysOld - Age of drafts to delete | |
* @returns {Promise<Object>} Cleanup results | |
*/ | |
async manualCleanupOldDrafts(daysOld?: number) { | |
this.logger.log( |
1 Data Flow/s detected
async manualDraftCleanup( |
await this.scheduledTasksService.manualCleanupOldDrafts(daysOld); |
async manualCleanupOldDrafts(daysOld?: number) { |
daysOld === undefined ? "" : ` (${daysOld} days old)` |
`Manual cleanup of old drafts requested${ |
this.logger.log( |
Secure Code Warrior Training Material
● Training
▪ Secure Code Warrior Log Forging Training
● Videos
▪ Secure Code Warrior Log Forging Video
● Further Reading
🏴 Suppress Finding
- ... as False Alarm
- ... as Acceptable Risk
PR Description
Overview:
Type of Issue:
feat
): New functionality or feature added.bug
): Issue or bug resolved.chore
): Maintenance, refactoring, or non-functional changes.doc
): Documentation improvements or additions.Change Type:
Testing & Validation:
Purpose:
Maintain connectivity with production database and backend server