Skip to content

GA 질의 이벤트 연결 키 충돌 해소 - #308

Open
goder-0 wants to merge 1 commit into
mainfrom
feature/#307-ga-query-id
Open

GA 질의 이벤트 연결 키 충돌 해소#308
goder-0 wants to merge 1 commit into
mainfrom
feature/#307-ga-query-id

Conversation

@goder-0

@goder-0 goder-0 commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

관련 이슈

PR 설명

GA 웹 태그가 내부 식별자로 사용하는 query_id와 애플리케이션의 질의 연결 키가 충돌해, 브라우저 이벤트만 서버 이벤트와 연결되지 않던 문제를 해결합니다.

변경 사항

  • query_submit, query_response_complete의 GA 파라미터를 app_query_id로 변경했습니다.
  • 정상·오류 응답 모두 두 서버 이벤트가 동일한 질의 UUID를 유지하도록 테스트를 보강했습니다.
  • GA payload에 기존 query_id가 남지 않는 것도 함께 검증합니다.
  • DB 컬럼과 API/WebSocket의 queryId 필드는 유지하며, 설명에서 GA 종속 표현만 제거했습니다.

리뷰 포인트

  • 프론트 PR #595와 함께 배포되어야 네 질의 이벤트가 동일한 app_query_id로 연결됩니다.
  • 데이터베이스 마이그레이션이나 API 계약 변경은 없습니다.

검증

  • RagChatServiceTest, Ga4MeasurementProtocolClientTest
  • Checkstyle main/test

@github-actions

github-actions Bot commented Sep 8, 2026

Copy link
Copy Markdown

📊 코드 커버리지 리포트

Overall Project 91.04% 🍏
Files changed 100% 🍏

File Coverage
RagChatService.java 97.7% 🍏

@goder-0
goder-0 requested a review from ckdals4600 September 9, 2026 04:43
@goder-0
goder-0 force-pushed the feature/#307-ga-query-id branch from c4145ee to 143f3e7 Compare September 9, 2026 08:57
@goder-0 goder-0 mentioned this pull request Sep 9, 2026
11 tasks
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.

[Analytics] GA 질의 이벤트 연결 키를 app_query_id로 변경

1 participant