fix: 모의서버 완화에 실효 도메인 검증 — #411 머지 레이스로 누락된 하드닝 회수 - #412
Merged
Conversation
use_mock=true라도 mock_url이 실전 도메인으로 오설정돼 있으면 KISApi가 그 URL을 base_url로 쓰므로 주문이 실서버로 간다 — 그 조합에서 평가 면제는 금지해야 한다. 완화 조건을 (use_mock AND 실효 mock_url에 'openapivts' 포함)으로 강화. 기괴한 오설정이지만 live gate는 가장 민감한 표면이라 싸게 막는다. 회귀 테스트 추가, 전체 1489 passed.
easygap
added a commit
that referenced
this pull request
Jul 6, 2026
fix: 모의서버 완화에 실효 도메인 검증 — #411 머지 레이스로 누락된 하드닝 회수
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
배경
#411의 마지막 하드닝 커밋(
44a8f31)이 머지와 푸시의 타이밍 레이스로 머지에서 빠졌습니다 — 이 PR이 그 커밋을 cherry-pick으로 회수합니다.내용 (fail-closed 하드닝)
use_mock=true라도mock_url이 실전 도메인으로 오설정돼 있으면 KISApi가 그 URL을base_url로 쓰므로 주문은 실서버로 가면서 게이트는 평가를 면제하는 조합이 가능했습니다. 완화 조건을 **(use_mock AND 실효 mock_url에openapivts포함)**으로 강화 — 기괴한 오설정이지만 live gate는 가장 민감한 표면이라 싸게 막습니다.검증
회귀 테스트 1건 추가(실전 도메인 오설정 → 완화 안 됨), 게이트 테스트 16건 통과. 전체 스위트 기준 1489 passed (#411 브랜치에서 검증 완료).