fix(tkuser): Sprint 001 리뷰 권장 개선 3건 — 방어 코딩 및 일관성 보완
- setLongServiceExclusion: affectedRows 체크 추가 (존재하지 않는 user_id → 404) - ACCESS_LEVELS: user: 1 키 추가 (role='user' 사용자 레벨 0 방지) - escapeHtml → escHtml 통일 (tkuser-vacations.js 라인 381) Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
@@ -130,10 +130,13 @@ async function setLongServiceExclusion(req, res, next) {
|
||||
}
|
||||
const { getPool } = require('../models/userModel');
|
||||
const db = getPool();
|
||||
await db.query(
|
||||
const [result] = await db.query(
|
||||
'UPDATE sso_users SET long_service_excluded = ? WHERE user_id = ?',
|
||||
[excluded ? 1 : 0, user_id]
|
||||
);
|
||||
if (result.affectedRows === 0) {
|
||||
return res.status(404).json({ success: false, error: '해당 사용자를 찾을 수 없습니다' });
|
||||
}
|
||||
res.json({ success: true, message: `장기근속 제외 설정이 ${excluded ? '활성화' : '해제'}되었습니다` });
|
||||
} catch (err) { next(err); }
|
||||
}
|
||||
|
||||
@@ -80,7 +80,7 @@ function requireAdminOrPermission(pageName) {
|
||||
* 최소 권한 레벨 체크 미들웨어
|
||||
* worker(1) < group_leader(2) < support_team(3) < admin(4) < system(5)
|
||||
*/
|
||||
const ACCESS_LEVELS = { worker: 1, group_leader: 2, support_team: 3, admin: 4, system: 5 };
|
||||
const ACCESS_LEVELS = { user: 1, worker: 1, group_leader: 2, support_team: 3, admin: 4, system: 5 };
|
||||
|
||||
function requireMinLevel(minLevel) {
|
||||
return (req, res, next) => {
|
||||
|
||||
Reference in New Issue
Block a user