Skip to content

fix: 코스 생성 시 GPS 경로 전체를 로그에 남기던 코드 제거#200

Open
unam98 wants to merge 1 commit into
mainfrom
fix/course-path-log-pii
Open

fix: 코스 생성 시 GPS 경로 전체를 로그에 남기던 코드 제거#200
unam98 wants to merge 1 commit into
mainfrom
fix/course-path-log-pii

Conversation

@unam98

@unam98 unam98 commented Jul 24, 2026

Copy link
Copy Markdown
Collaborator

작업 배경

  • prod에 OTel 로그 수집을 붙이기 전 로깅 코드 PII 감사 중 발견
  • CoordinatePathConverter.getLineString()이 매 코스 생성마다 전체 GPS 좌표를 INFO 레벨로 로깅하고 있었음 — 러닝 경로는 집/직장 위치 추정이 가능한 민감 데이터라, 외부 로그 수집(Grafana Cloud)으로 나가면 안 됨

변경 사항

영역 내용
CoordinatePathConverter.java 정상 경로에서 좌표 전체를 찍던 log.info("create course!") / log.info(sb.toString()) 제거. 검증 실패 시 로그도 원본 좌표 대신 size/타입/에러 메시지만 남기도록 변경

영향 범위

  • 로깅 내용만 변경, 코스 생성/변환 로직 자체는 동일 (좌표 → LineString 변환 결과 불변)
  • 런타임 동작 영향 없음

Test Plan

  • ./gradlew compileJava 통과 확인
  • dev 배포 후 코스 생성 API 정상 동작 확인

🤖 Generated with Claude Code

- getLineString()이 매 코스 생성마다 좌표 전체를 INFO 레벨로 로깅하던 부분 제거
- 검증 실패 시에도 원본 좌표 대신 size/타입/에러 메시지만 로깅하도록 변경
- prod에 OTel 로그 수집을 붙이기 전 필수 정리 (러닝 경로는 위치 추정이 가능한 민감 데이터)
@coderabbitai

coderabbitai Bot commented Jul 24, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: fe611935-b5ef-4e3b-9eb9-08ac396f94d8

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/course-path-log-pii

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@unam98 unam98 self-assigned this Jul 24, 2026
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