[m-a-king]

한눈에

최종 주소가 http 여도 상품 정체성 확정

• 이전: 추출 서비스(extractor)가 준 최종 주소가 http 면 core 가 상품 정체성 확정을 건너뜀
  • 원인: core 가 최종 주소를 등록 링크와 같은 규칙(https 만)으로 읽음
• 추출 서비스가 http 로 넘어가는 이동도 따라가게 바뀜
• core 가 최종 주소는 http·https 를 모두 받음
  • 사용자가 등록하는 링크는 그대로 https 만
• 결과: 같은 주소의 http·https 가 같은 상품 정체성으로 모임
• 머지하면 dev 에 자동 배포. prod 는 승격 때 반영

Situation

원인 추가

• 파싱 귀결점의 http 주소 허용 #1238
• 추가
  • core 는 최종 주소로 직접 요청을 보내지 않음. 정규화해 DB 에 넣기만 함
    • 결과: http 를 받아도 core 쪽 요청 위험 없음. 내부 주소 검사는 extractor 가 매 홉에서 끝냄

Task

범위

• 파싱 귀결점의 http 주소 허용 #1238

Action

최종 주소 전용 파싱 추가

• http·https 를 받고 host 가 없으면 거부
  • 원인: 정규화가 host 를 요구함. host 없는 주소를 파싱에서 걸러 "확정 건너뜀" 경로로 보냄
• 등록 링크 파싱은 그대로 https 만
• 정체성 기록이 최종 주소를 이 파싱으로 읽음
• 이름과 받는 기준을 extractor 의 홉 주소 파싱(parseHop)과 같게 둠

선택지	최종 주소의 http	비용
최종 주소 전용 파싱 추가 (채택)	정체성 확정	파싱 함수 둘
등록 링크 파싱도 http 허용	정체성 확정	사용자의 http 링크 등록 허용
그대로 둠	확정 건너뜀	http 로 끝나는 상품의 정체성 공유 없음

테스트 예시 교체

• 파싱 불가 예시를 http 에서 ftp 로 교체
  • 원인: http 가 이제 파싱되어 예시가 의도를 잃음

Result

검증

• 정체성 기록을 등록 링크 파싱으로 되돌리면 새 통합 테스트 1건(http 최종 주소의 정체성)이 실패함
• 위험: 같은 host·path 의 http·https 페이지가 서로 다른 상품이면 하나로 묶임
  • 정규화가 이미 scheme 을 https 로 다시 조립하던 가정과 같음

후속

• 머지 순서는 SSRF PR 과 무관
  • SSRF 검사를 직접 가져오기 한 곳으로 정리 extractor#112 의 배포보다 이 PR 의 prod 승격이 먼저면 공백 없음
  • 늦으면 그 사이 http 최종 주소의 정체성 확정만 건너뜀

---

연관 이슈

• close 파싱 귀결점의 http 주소 허용 #1238

[coderabbitai]

Important

• 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

• Configuration used: Repository: TeamPiKi/core/.coderabbit.yml
• Review profile: CHILL
• Plan: Advanced
• Run ID: 5ae21798-5469-4b35-b83f-cc531cafc926

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[github-actions]

Discord 스레드 연동용 메타데이터입니다. discord-pr-bot 워크플로가 자동 생성하며, 수정·삭제하면 PR 과 Discord 알림 연동이 끊깁니다.