Skip to content

[Feat] 저장된 URL 중복 체크 API 추가 - #43

Merged
sunm2n merged 4 commits into
devfrom
feat/#42-check_same_url
May 27, 2026
Merged

[Feat] 저장된 URL 중복 체크 API 추가#43
sunm2n merged 4 commits into
devfrom
feat/#42-check_same_url

Conversation

@sunm2n

@sunm2n sunm2n commented May 26, 2026

Copy link
Copy Markdown
Contributor

요약

FastAPI 분석 요청 전, 사용자가 입력한 URL이 이미 저장된 링크에 존재하는지 확인하는 API를 추가합니다.

  • GET /api/v1/saved-links/check?url={url}
  • Analysis.original_url 기준 exact match로 현재 멤버의 저장 링크 중복 여부 반환
  • url 파라미터 누락 / 빈 문자열 시 400 처리

테스트

중복 URL 체크 테스트

image

연관 이슈 및 Close 할 이슈 작성

close #42

Pull Request 체크리스트

TODO

  • 최종 결과물을 확인했는가?
  • 의미 있는 커밋 메시지를 작성했는가?

sunm2n added 4 commits May 26, 2026 12:18
- SavedLinkServiceTest: checkUrl 존재/미존재 케이스 검증
- SavedLinkControllerTest: GET /saved-links/check 응답 검증

IssueNum #42
- GET /api/v1/saved-links/check?url={url}
- Analysis.originalUrl 기준 exact match로 멤버별 중복 여부 확인

IssueNum #42
- MissingServletRequestParameterException → 400 처리 (GlobalExceptionHandler)
- 빈 문자열 url 파라미터 → 400 처리 (컨트롤러 직접 검증)

IssueNum #42
- 400, 401 응답 코드 추가

IssueNum #42
@sunm2n
sunm2n requested review from kbh0218 and minsoo0506 May 26, 2026 06:18
@sunm2n sunm2n self-assigned this May 26, 2026
@sunm2n sunm2n added the feature 새로운 기능/특징 label May 26, 2026

@minsoo0506 minsoo0506 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

GET /api/v1/saved-links/check?url=...에서 빈 문자열/누락 파라미터를 400으로 처리하고, 현재 멤버의 저장 링크 중 Analysis.originalUrl exact match 여부를 확인하는 구현도 PR 의도와 일치합니다.

수정할 것이 없어 보입니다. 고생하셨습니다 :)

@sunm2n
sunm2n merged commit 1ced4e6 into dev May 27, 2026
@sunm2n
sunm2n deleted the feat/#42-check_same_url branch May 27, 2026 03:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature 새로운 기능/특징

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE] 동일 URL 확인 API 개발

2 participants