open-source/qinglong
1
0
Fork 0
mirror of https://github.com/whyour/qinglong.git synced 2026-06-28 02:45:08 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix: 修复鉴权绕过修复中的两个 bug

Browse Source 
1. 路径规范化中间件只检查 API/OPEN 前缀大小写而非整个路径
   避免误拦截 /api/scripts/MyScript.js 等含大写的合法请求
2. init 中间件改用 pathLower 进行比较，补全防御深度

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
This commit is contained in:
homePC 2026-05-21 14:57:19 +08:00
parent 66f9457be8
commit 1d5f23bc60
1 changed files with 8 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
back/loaders/express.ts
View File

@ -21,18 +21,17 @@ export default ({ app }: { app: Application }) => {
// Security: Path normalization middleware to prevent case variation attacks // Security: Path normalization middleware to prevent case variation attacks
app.use((req, res, next) => { app.use((req, res, next) => {
const originalPath = req.path; // Only check the API/OPEN prefix for case variations, not the entire path.
const normalizedPath = originalPath.toLowerCase(); // This blocks /API/..., /Api/..., /OPEN/..., /Open/... bypass attempts
// while allowing legitimate mixed-case paths like /api/scripts/MyScript.js
// Block requests with case variations on protected paths const pathLower = req.path.toLowerCase();
if (originalPath !== normalizedPath && if ((pathLower.startsWith('/api/') && !req.path.startsWith('/api/')) ||
(normalizedPath.startsWith('/api/') || normalizedPath.startsWith('/open/'))) { (pathLower.startsWith('/open/') && !req.path.startsWith('/open/'))) {
return res.status(400).json({ return res.status(400).json({
code: 400, code: 400,
message: 'Invalid path format' message: 'Invalid path format',
}); });
} }
next(); next();
}); });
@ -126,7 +125,7 @@ export default ({ app }: { app: Application }) => {
'/api/user/notification/init', '/api/user/notification/init',
'/open/user/init', '/open/user/init',
'/open/user/notification/init', '/open/user/notification/init',
].includes(req.path) ].includes(pathLower)
) { ) {
return next(); return next();
} }