Skip to content

링크 저장 URL 정규화 및 검증 적용 - #300

Merged
goder-0 merged 1 commit into
mainfrom
feature/issue-298-url-normalization
Sep 1, 2026
Merged

링크 저장 URL 정규화 및 검증 적용#300
goder-0 merged 1 commit into
mainfrom
feature/issue-298-url-normalization

Conversation

@goder-0

@goder-0 goder-0 commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

관련 이슈

PR 설명

  • 공통 UrlNormalizer를 추가해 앞뒤 공백과 줄바꿈을 제거하고, 중복된 HTTP/HTTPS 프로토콜을 하나로 정리합니다.
  • 프로토콜이 없는 URL에는 https://를 보완하고 HTTP/HTTPS 외 프로토콜과 점이 없는 도메인을 거부합니다.
  • 링크 생성과 URL 중복 확인이 동일한 정규화 결과를 사용하도록 LinkService에 적용했습니다.
  • 메타데이터 수집은 정규화된 URL을 대상으로 기존 SSRF 검증을 실행한 뒤 크롤링합니다.
  • http 프로토콜과 URL의 path, query, fragment는 그대로 보존합니다.

검증

  • URL 정규화, 링크 생성·중복 확인, 메타데이터 수집 전 SSRF 검증 단위 테스트 통과
  • main/test 체크스타일 통과
  • 전체 테스트는 로컬 테스트 환경의 AWS 설정 부재로 ApplicationContext 기반 테스트가 시작되지 못했으며, 변경 관련 테스트는 모두 통과했습니다.

@github-actions

github-actions Bot commented Aug 24, 2026

Copy link
Copy Markdown

📊 코드 커버리지 리포트

Overall Project 91.04% -0.07% 🍏
Files changed 97.04% 🍏

File Coverage
LinkService.java 100% 🍏
UrlNormalizer.java 96.51% -3.49% 🍏

Comment thread src/main/java/com/sofa/linkiving/domain/link/util/UrlNormalizer.java Outdated
Comment thread src/main/java/com/sofa/linkiving/domain/link/util/OgTagCrawler.java Outdated
@goder-0
goder-0 force-pushed the feature/issue-298-url-normalization branch 2 times, most recently from e826212 to 9efce97 Compare August 26, 2026 12:04
@goder-0
goder-0 force-pushed the feature/issue-298-url-normalization branch from 9efce97 to 42425ed Compare September 1, 2026 08:34
@goder-0
goder-0 merged commit 66e1aca into main Sep 1, 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.

링크 저장 URL 정규화 및 검증 적용

2 participants