Skip to content

fix: www·루트 슬래시 차이로 인한 URL 중복 저장 방지 (#333) - #334

Merged
goder-0 merged 2 commits into
mainfrom
feature/333-root-url-dedup
Oct 18, 2026
Merged

goder-0 merged 2 commits into
mainfrom
feature/333-root-url-dedup

Conversation

@goder-0

@goder-0 goder-0 commented Oct 18, 2026 •

Copy link
Copy Markdown
Contributor

관련 이슈

Closes #333
Refs #324

PR 설명

naver.com, naver.com/, www.naver.com, www.naver.com/을 같은 링크로 인식하고 신규 DB 저장값을 https://naver.com으로 통일합니다. 중복 조회와 생성에 같은 기준을 적용하며, 기존 DB의 www·루트 슬래시 표기도 함께 비교해 재저장을 차단합니다.

메타정보 수집에는 입력한 호스트를 사용합니다. m.·blog. 등 다른 서브도메인과 명시적 HTTP/HTTPS, 포트·경로·쿼리·fragment는 구분하며 일반 경로의 끝 슬래시는 유지합니다. 이미 중복 저장된 데이터의 병합·삭제나 스키마 변경은 없습니다. CHANGELOG의 Unreleased / Fixed에 반영했습니다.

검증

Java 17에서 build jacocoTestCoverageVerification 통과(테스트 584건, Checkstyle 및 커버리지 게이트 포함). H2 통합 테스트로 네 표기의 16가지 저장·입력 조합, 정규화된 DB 저장값, 회원·삭제 링크 제외와 기존 중복 데이터의 일관된 조회를 확인했습니다. 운영 PostgreSQL 검증은 별도입니다.

병합 검토

최종 head 5ff4473에서 CI·배포 스크립트 검사·로컬 번들 빌드가 모두 통과했습니다. 요청자의 명시적인 관리자 우회 승인에 따라 팀원 승인 1건 요건을 이번 병합에 한해 우회하고 rebase merge합니다. 저장소 규칙·기존 데이터·릴리즈 태그는 변경하지 않습니다.

@github-actions

github-actions Bot commented Oct 18, 2026 •

Copy link
Copy Markdown

📊 코드 커버리지 리포트

Overall Project 91.84% 🍏
Files changed 100% 🍏

File Coverage
LinkService.java 100% 🍏
UrlNormalizer.java 97.75% 🍏

@goder-0 goder-0 changed the title fix: 루트 URL 끝 슬래시 중복 저장 방지 (#333) fix: www·루트 슬래시 차이로 인한 URL 중복 저장 방지 (#333) Oct 18, 2026
@goder-0
goder-0 merged commit d8c600b into main Oct 18, 2026
3 checks passed
@goder-0 goder-0 mentioned this pull request Oct 19, 2026
13 tasks done
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.

www·루트 슬래시 차이로 발생하는 URL 중복 저장 방지

1 participant