Skip to content

두 개 이상의 에이전트가 마지막 생성된 에이전트의 token, app key, app secret 을 사용하는 문제 수정 - #53

Open
ssethj wants to merge 1 commit into
unohee:mainfrom
ssethj:fix/multi-agent-auth-leak
Open

두 개 이상의 에이전트가 마지막 생성된 에이전트의 token, app key, app secret 을 사용하는 문제 수정#53
ssethj wants to merge 1 commit into
unohee:mainfrom
ssethj:fix/multi-agent-auth-leak

Conversation

@ssethj

@ssethj ssethj commented Jul 28, 2026

Copy link
Copy Markdown

서로 다른 CANO (계좌 앞 8자리) 를 갖는 KIS Agent 를 두 개 이상 사용할 때 모든 에이전트가 가장 마지막으로 생성된 에이전트의 토큰, app key, app secret 을 사용하는 문제를 수정했습니다.

@ssethj ssethj changed the title Fix token/key/secret leak with multiple agents 두 개 이상의 에이전트가 마지막 생성된 에이전트의 token, app key, app secret 을 사용하는 문제 수정 Jul 28, 2026
@ssethj
ssethj force-pushed the fix/multi-agent-auth-leak branch from aff33ff to 46a1aea Compare August 4, 2026 01:55

@unohee unohee left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

방향 자체는 맞지만 현재 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로 실패했습니다. 실패 체크가 해소되면 다시 확인하겠습니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants