hanh.tonguyen/paperclip
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(server): use realpathSync for .env path dedup to handle symlinks

Browse Source 
realpathSync resolves symlinks and normalizes case, preventing
double-loading the same .env file when paths differ only by
symlink indirection or filesystem case.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
Matt Van Horn
2026-03-14 09:05:51 -07:00
parent 303c00b61b
commit ff4f326341
1 changed files with 5 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
server/src/config.ts
View File

@@ -1,5 +1,5 @@
import { readConfigFile } from "./config-file.js"; import { readConfigFile } from "./config-file.js";
import { existsSync } from "node:fs"; import { existsSync, realpathSync } from "node:fs";
import { resolve } from "node:path"; import { resolve } from "node:path";
import { config as loadDotenv } from "dotenv"; import { config as loadDotenv } from "dotenv";
import { resolvePaperclipEnvPath } from "./paths.js"; import { resolvePaperclipEnvPath } from "./paths.js";
@@ -29,7 +29,10 @@ if (existsSync(PAPERCLIP_ENV_FILE_PATH)) {
} }
const CWD_ENV_PATH = resolve(process.cwd(), ".env"); const CWD_ENV_PATH = resolve(process.cwd(), ".env");
if (CWD_ENV_PATH !== PAPERCLIP_ENV_FILE_PATH && existsSync(CWD_ENV_PATH)) { const isSameFile = existsSync(CWD_ENV_PATH) && existsSync(PAPERCLIP_ENV_FILE_PATH)
? realpathSync(CWD_ENV_PATH) === realpathSync(PAPERCLIP_ENV_FILE_PATH)
: CWD_ENV_PATH === PAPERCLIP_ENV_FILE_PATH;
if (!isSameFile && existsSync(CWD_ENV_PATH)) {
loadDotenv({ path: CWD_ENV_PATH, override: false, quiet: true }); loadDotenv({ path: CWD_ENV_PATH, override: false, quiet: true });
} }