pjandro/OmniRoute
1
0
Fork 0
mirror of https://github.com/diegosouzapw/OmniRoute.git synced 2026-07-26 09:52:11 +03:00
Code Issues Packages Projects Releases Wiki Activity

fix: handle EACCES on restricted home directories (#133)

Browse Source 
Wrap fs.mkdirSync(DATA_DIR) in try/catch so OmniRoute doesn't crash
when the user's home directory is not writable (e.g. restricted
environments, npm global install with different user).
Prints a clear warning with DATA_DIR env var recommendation.
This commit is contained in:
diegosouzapw
2026-02-25 18:41:54 -03:00
parent 5a65585c16
commit 62069dac98
1 changed files with 11 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
src/lib/db/core.ts
View File

@@ -24,9 +24,18 @@ export const SQLITE_FILE = isCloud ? null : path.join(DATA_DIR, "storage.sqlite"
const JSON_DB_FILE = isCloud ? null : path.join(DATA_DIR, "db.json");
export const DB_BACKUPS_DIR = isCloud ? null : path.join(DATA_DIR, "db_backups");
// Ensure data directory exists
// Ensure data directory exists — with fallback for restricted home directories (#133)
if (!isCloud && !fs.existsSync(DATA_DIR)) {
fs.mkdirSync(DATA_DIR, { recursive: true });
try {
fs.mkdirSync(DATA_DIR, { recursive: true });
} catch (err: unknown) {
const msg = err instanceof Error ? err.message : String(err);
console.warn(
`[DB] Cannot create data directory '${DATA_DIR}': ${msg}\n` +
`[DB] Set the DATA_DIR environment variable to a writable path, e.g.:\n` +
`[DB] DATA_DIR=/path/to/writable/dir omniroute`
);
}
}
// ──────────────── Schema ────────────────