refactor(backup): convert sync fs operations to async promises #235

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

Closed

BillChirico wants to merge 9 commits into main from feat/error-handling-audit

Closed

refactor(backup): convert sync fs operations to async promises #235

fix(backup): await async operations in API routes

GitHub Advanced Security / CodeQL failed Mar 3, 2026 in 2s

3 new alerts including 3 high severity security vulnerabilities

New alerts in code changed by this pull request

Security Alerts:

3 high

See annotations below for details.

View all branch alerts.

Annotations

Check failure on line 290 in src/modules/backup.js

Code scanning / CodeQL

Uncontrolled data used in path expression High

This path depends on a

.
This path depends on a

.

Check failure on line 295 in src/modules/backup.js

Code scanning / CodeQL

Potential file system race condition High

The file may have changed since it

.

Check failure on line 295 in src/modules/backup.js

Code scanning / CodeQL

Uncontrolled data used in path expression High

This path depends on a

.
This path depends on a

.