두 개 이상의 에이전트가 마지막 생성된 에이전트의 token, app key, app secret 을 사용하는 문제 수정 - #53
Open
ssethj wants to merge 1 commit into
Open
두 개 이상의 에이전트가 마지막 생성된 에이전트의 token, app key, app secret 을 사용하는 문제 수정#53ssethj wants to merge 1 commit into
ssethj wants to merge 1 commit into
Conversation
ssethj
force-pushed
the
fix/multi-agent-auth-leak
branch
from
August 4, 2026 01:55
aff33ff to
46a1aea
Compare
unohee
requested changes
Aug 10, 2026
unohee
left a comment
Owner
There was a problem hiding this comment.
방향 자체는 맞지만 현재 merge commit 기준 회귀가 있어 머지를 보류합니다.
coverage-analysis가 실제로 실패 중이며, tests/unit/test_client_extra_paths.py의 2개 테스트에서 _auth_headers()가 self.token을 직접 읽다가 AttributeError가 발생합니다. 해당 테스트의 경량 클라이언트처럼 생성자 전체를 거치지 않은 호환 경로도 기존에는 동작했습니다. getattr(self, "token", None)처럼 안전하게 읽거나 테스트 헬퍼/계약을 명시적으로 정리해야 합니다.
또한 이 PR의 핵심 요구사항인 “서로 다른 자격증명을 가진 두 클라이언트를 번갈아 호출해도 각자의 token/app key/app secret을 유지한다”는 회귀 테스트가 없습니다. 그 테스트를 추가하면 수정 목적과 토큰 재발급 후 헤더 갱신까지 훨씬 확실하게 보장할 수 있습니다.
현재 상태: 일반 CI의 lint/type/test/quality gate는 통과했지만 coverage-analysis는 2 failed, 2134 passed로 실패했습니다. 실패 체크가 해소되면 다시 확인하겠습니다.
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
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.
서로 다른
CANO(계좌 앞 8자리) 를 갖는 KIS Agent 를 두 개 이상 사용할 때 모든 에이전트가 가장 마지막으로 생성된 에이전트의 토큰, app key, app secret 을 사용하는 문제를 수정했습니다.