Skip to content

fix(messaging): publishToAi 시 traceId MDC 누락 시 새 UUID 자동 생성#39

Merged
i3months merged 1 commit into
devfrom
fix/publisher-trace-id-fallback
May 31, 2026
Merged

fix(messaging): publishToAi 시 traceId MDC 누락 시 새 UUID 자동 생성#39
i3months merged 1 commit into
devfrom
fix/publisher-trace-id-fallback

Conversation

@i3months

Copy link
Copy Markdown
Member

운영 검증 중 발견. callback handler 흐름에서 자동 발화된 SessionEndedEvent → SessionFeedbackRequester.publishToAi 호출 시 MDC 에 traceId 가 없어 envelope.traceId=null 로 발행. AI 측 Envelope[T] 의 traceId 가 string required 라 Pydantic ValidationError ('Input should be a valid string [...] input_value=None') 로 메시지 거절.

해결: publishToAi 내부에 fallback — MDC traceId null/blank 면 UUID.randomUUID() 채움. JwtAuthenticationFilter 흐름은 MDC 가 채워져 있고, RabbitListener/TransactionalEventListener 흐름은 이 fallback 으로 균일하게 traceId 보장.

증상: 운영 골든패스 (Sprint3 verify, sessionId=4) 에서

  • SessionFeedbackRequester: generate.feedback published. (정상)
  • feedback consumer: feedback.parse.failed traceId: Input should be a valid string
  • /api/sessions/4/feedback 무한 FEEDBACK_NOT_READY

변경 사항

운영 검증 중 발견. callback handler 흐름에서 자동 발화된 SessionEndedEvent →
SessionFeedbackRequester.publishToAi 호출 시 MDC 에 traceId 가 없어 envelope.traceId=null
로 발행. AI 측 Envelope[T] 의 traceId 가 string required 라 Pydantic ValidationError
('Input should be a valid string [...] input_value=None') 로 메시지 거절.

해결: publishToAi 내부에 fallback — MDC traceId null/blank 면 UUID.randomUUID() 채움.
JwtAuthenticationFilter 흐름은 MDC 가 채워져 있고, RabbitListener/TransactionalEventListener
흐름은 이 fallback 으로 균일하게 traceId 보장.

증상: 운영 골든패스 (Sprint3 verify, sessionId=4) 에서
  - SessionFeedbackRequester: generate.feedback published. (정상)
  - feedback consumer: feedback.parse.failed traceId: Input should be a valid string
  - /api/sessions/4/feedback 무한 FEEDBACK_NOT_READY
@vercel

vercel Bot commented May 31, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
stackup Ready Ready Preview, Comment May 31, 2026 1:56pm

@i3months i3months merged commit e2023c9 into dev May 31, 2026
3 checks 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.

1 participant