feat/security-hardening #5

gahusb · 2026-05-17T13:59:51+09:00

gahusb commented

2026-05-17 13:59:51 +09:00

No description provided.

gahusb added 3 commits 2026-05-17 13:59:53 +09:00

fix(packs-lab): sign-link path traversal — startswith → relative_to (CODE_REVIEW F1) 1e5e1bcdff

str(abs_path).startswith(str(PACK_HOST_DIR))는 trailing slash가 없어
sibling 경로(/foo/packs ↔ /foo/packs_evil)를 통과시켜 DSM API에 잘못된
호스트 경로를 전달할 수 있었음. Path.relative_to 기반으로 컴포넌트 단위
엄격 검증으로 교체. test_sign_link_rejects_sibling_path 회귀 테스트
추가 (RED → GREEN 검증).

fix(stock): admin API auth hardening — ADMIN_API_KEY 빈 값 시 503 거부 (CODE_REVIEW F2) 6053e69afc

운영 .env에 ADMIN_API_KEY가 누락되면 verify_admin이 무조건 통과해서
/api/trade/balance, /api/trade/order 인증이 무력화되던 문제 차단.

- ADMIN_API_KEY 설정 + 올바른 키 → 통과 (기존 동작)
- ADMIN_API_KEY 설정 + 잘못된 키 → 401 (기존 동작)
- ADMIN_API_KEY 미설정 + ALLOW_UNAUTHENTICATED_ADMIN=true → 통과 (dev mode)
- ADMIN_API_KEY 미설정 + dev flag 없음 → 503 (신규, 운영 보호)

.env.example에 신규 ALLOW_UNAUTHENTICATED_ADMIN=false 안내 추가.
stock/pytest.ini 신규 (pythonpath=. 설정으로 tests 모듈 import 가능).
test_admin_auth.py 4 케이스 (RED → GREEN 검증, regression 포함).

docs(env): add ALLOW_UNAUTHENTICATED_ADMIN guidance for F2 49c5c57be5

gahusb merged commit faffca0967 into main

2026-05-17 14:00:04 +09:00

gahusb referenced this issue from a commit

2026-05-17 14:00:06 +09:00

Merge pull request 'feat/security-hardening' (#5) from feat/security-hardening into main

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: gahusb/web-page-backend#5