Skip to content

[FEATURE] 페이지 불가용(PAGE_UNAVAILABLE) 처리 추가 - #47

Merged
sunm2n merged 1 commit into
devfrom
feat/#46-not-found-page-case
May 31, 2026
Merged

[FEATURE] 페이지 불가용(PAGE_UNAVAILABLE) 처리 추가#47
sunm2n merged 1 commit into
devfrom
feat/#46-not-found-page-case

Conversation

@sunm2n

@sunm2n sunm2n commented May 31, 2026

Copy link
Copy Markdown
Contributor

요약

URL 검사 콜백 흐름에서 FastAPI가 전달하는 stages.content_analysisstatusCode, error, reason 필드를 Spring이 역직렬화하지 못하던 문제를 수정하고, 페이지 불가용 사유를 클라이언트 응답에 노출합니다.

  • ContentAnalysisStagestatusCode, error, reason 필드 추가
  • AnalysisResponsecontentAnalysisError 필드 추가 — fetched=false일 때만 JSON에 노출 (@JsonInclude(NON_NULL))
  • 기존 정상 fetch 케이스의 응답 구조는 변경 없음

연관 이슈 및 Close 할 이슈 작성

close #46

Pull Request 체크리스트

TODO

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

- ContentAnalysisStage에 statusCode, error, reason 필드 추가
- AnalysisResponse forSucceeded()에서 fetched=false 시 contentAnalysisError 노출

IssueNum #46
@sunm2n sunm2n changed the title [Feat] 페이지 불가용(PAGE_UNAVAILABLE) 처리 추가 [FEATURE] 페이지 불가용(PAGE_UNAVAILABLE) 처리 추가 May 31, 2026
@sunm2n
sunm2n requested a review from minsoo0506 May 31, 2026 08:16
@sunm2n sunm2n self-assigned this May 31, 2026
@sunm2n sunm2n added the feature 새로운 기능/특징 label May 31, 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.

넵 확인했습니다! 코멘트 남길만한 부분이 없어서 바로 머지하셔도 될 것 같습니다! 고생하셨습니다 👍

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] 페이지 불가용(PAGE_UNAVAILABLE) 처리 추가

2 participants