本文详解为何基于 lastIndexOf('.') 提取扩展名的校验逻辑在 Windows 上失效，并提供跨平台兼容的标准化处理方案，确保 .exe、.bat 等危险文件类型在所有操作系统中均被可靠拦截。
本文详解为何基于 `lastindexof('.')` 提取扩展名的校验逻辑在 windows 上失效，并提供跨平台兼容的标准化处理方案，确保 `.exe`、`.bat` 等危险文件类型在所有操作系统中均被可靠拦截。
在 Node.js + Express + Multer 的文件上传场景中，开发者常通过解析文件名后缀来实现扩展名白/黑名单校验。然而，如问题所示：同一段校验逻辑（如禁止 .exe、.bat）在 macOS 上生效，却在 Windows 上失效——用户仍可成功上传可执行文件。根本原因在于
Windows 文件系统对文件名中点号（.）的处理更宽松，且原始代码中扩展名提取逻辑存在严重缺陷
。
❌ 原始逻辑的问题分析
关键错误出现在这行：
该写法试图“跳过最后一个点前的字符”，但存在三重隐患：
位运算滥用
：>>> 0 强制转为无符号整数，当 lastIndexOf('.') 返回 -1（即无点号）时，(-1 - 1) >>> 0 得到 4294967294，导致 slice(4294967294 + 2) 返回空字符串，后续 includes('') 恒为 true —— 这会意外放行所有文件；
大小写不敏感缺失
：Windows 文件系统默认不区分大小写（如 setup.EXE），但 '.exe'.includes('EXE') 为 false；
扩展名格式不规范
：未统一补全前导点号，导致 includes('.exe') 匹配失败（例如提取出 'exe' 而非 '.exe'）。
✅ 正确的跨平台扩展名提取与校验
应使用标准、健壮的方式获取并标准化扩展名：
? 补充建议与最佳实践
优先使用 path.extname()（Node.js 内置）
：更可靠，自动处理边界情况（如 file. 或 .gitignore）：
增强防御：结合 MIME 类型校验
扩展名可伪造，务必配合 file.mimetype（如 application/x-msdownload）进行二次验证：
日志审计
：记录被拦截的文件名与原始扩展名，便于安全追踪：
总结
：文件扩展名校验不是简单字符串操作，而是涉及路径解析、大小写归一化、边界容错的系统性任务。务必使用 path.extname() 或手动实现带空值防护的提取逻辑，并始终对扩展名执行 .toLowerCase() 与前导点标准化，才能真正实现跨平台、防绕过的安全控制。
const fileExtension = fileName
.slice(((fileName.lastIndexOf('.') - 1) >>> 0) + 2);async upload(
@UploadedFile() file: Express.Multer.File,
@Body() body: FileUploadDto,
) {
const forbiddenExt = ['.exe', '.bat', '.cmd', '.ps1'];
const fileName = file.filename || file.originalname;
// ✅ 安全提取扩展名：先找最后一个点，再截取子串，转小写，补前导点
const lastDotIndex = fileName.lastIndexOf('.');
const fileExtension = lastDotIndex === -1
? ''
: fileName.slice(lastDotIndex).toLowerCase(); // 保留点号，如 '.exe'
// ✅ 标准化：确保以点开头，避免 'exe' 和 '.exe' 匹配不一致
const normalizedExtension = fileExtension.startsWith('.')
? fileExtension
: `.${fileExtension}`;
if (forbiddenExt.includes(normalizedExtension)) {
throw new HttpException(
'File type is not supported. Upload only .apk or .ipa',
HttpStatus.BAD_REQUEST,
);
}
const response = await this.uploadFile(body.filename, file.buffer);
return { success: true };
}import * as path from 'path';
const ext = path.extname(fileName).toLowerCase(); // 自动返回 '.exe'const dangerousMimes = ['application/x-msdownload', 'application/x-bat'];
if (dangerousMimes.includes(file.mimetype)) {
throw new HttpException('Suspicious MIME type detected', HttpStatus.BAD_REQUEST);
}console.warn(`Blocked upload: ${fileName} -> extension="${normalizedExtension}"`);