Update Dockerfile to create both data and logs directories; enhance logger to handle directory and file creation errors gracefully. Log warnings for permission issues without blocking application startup.

This commit is contained in:
theluyuan 2025-10-31 09:33:05 +08:00
parent f96a69e846
commit c9ccb3435d
2 changed files with 35 additions and 13 deletions

View File

@ -31,8 +31,8 @@ RUN pnpm i
# 复制项目文件 # 复制项目文件
COPY . . COPY . .
# 创建数据目录并设置权限 # 创建数据目录和日志目录并设置权限
RUN mkdir -p /app/data && \ RUN mkdir -p /app/data /app/logs && \
chown -R node:node /app chown -R node:node /app
# 切换到非root用户 # 切换到非root用户

View File

@ -28,10 +28,16 @@ class Logger {
} }
ensureLogsDirectory() { ensureLogsDirectory() {
if (!fs.existsSync(this.logsDir)) { try {
fs.mkdirSync(this.logsDir, { recursive: true }); if (!fs.existsSync(this.logsDir)) {
// 使用原始console避免循环调用 fs.mkdirSync(this.logsDir, { recursive: true });
this.originalConsole.log(`创建日志目录: ${this.logsDir}`); // 使用原始console避免循环调用
this.originalConsole.log(`创建日志目录: ${this.logsDir}`);
}
} catch (error) {
// 如果创建目录失败(比如权限问题),警告但不阻止应用启动
this.originalConsole.warn(`无法创建日志目录 ${this.logsDir}:`, error.message);
this.originalConsole.warn('日志将仅输出到控制台,不会保存到文件');
} }
} }
@ -48,9 +54,15 @@ class Logger {
this.currentDate = today; this.currentDate = today;
this.currentLogFile = this.getLogFileName(); this.currentLogFile = this.getLogFileName();
// 确保文件存在 // 确保文件存在 - 使用try-catch处理权限问题
if (!fs.existsSync(this.currentLogFile)) { if (!fs.existsSync(this.currentLogFile)) {
fs.writeFileSync(this.currentLogFile, ''); try {
// 尝试创建空文件,如果失败则会在下次写入时自动创建
fs.writeFileSync(this.currentLogFile, '', { flag: 'a' });
} catch (error) {
// 如果创建文件失败(比如权限问题),不抛出错误,会在后续写入时处理
this.originalConsole.warn(`无法创建日志文件 ${this.currentLogFile}:`, error.message);
}
} }
} }
} }
@ -71,8 +83,15 @@ class Logger {
const timestamp = this.formatTimestamp(); const timestamp = this.formatTimestamp();
const logLine = `[${timestamp}] [${level.toUpperCase()}] ${message}\n`; const logLine = `[${timestamp}] [${level.toUpperCase()}] ${message}\n`;
// 写入文件 // 写入文件 - 使用flag: 'a'确保文件不存在时自动创建
fs.appendFileSync(this.currentLogFile, logLine, 'utf8'); try {
fs.appendFileSync(this.currentLogFile, logLine, { encoding: 'utf8', flag: 'a' });
} catch (error) {
// 如果写入失败比如权限问题输出到原始console
this.originalConsole.error('写入日志文件失败:', error.message);
this.originalConsole[level](message);
return; // 不继续处理
}
// 添加到缓冲区用于实时推送 // 添加到缓冲区用于实时推送
const logEntry = { const logEntry = {
@ -102,9 +121,12 @@ class Logger {
} }
} catch (error) { } catch (error) {
// 如果文件写入失败至少输出到原始console // 如果文件写入失败(已经在上面处理了),这里只是防止其他错误
this.originalConsole.error('写入日志文件失败:', error); // 如果上面的appendFileSync已经捕获了错误这里不会执行
this.originalConsole[level](message); // 这里主要是处理其他可能的错误
if (error.code !== 'EACCES' && !error.message.includes('permission')) {
this.originalConsole.error('日志处理失败:', error);
}
} }
} }