Skip to content

홉 상한 도달 렌더 확정 실패 전환 - #68

Open
m-a-king wants to merge 1 commit into
mainfrom
fix/hop-cap-too-many-redirects
Open

m-a-king wants to merge 1 commit into
mainfrom
fix/hop-cap-too-many-redirects

Conversation

@m-a-king

@m-a-king m-a-king commented Sep 10, 2026

Copy link
Copy Markdown
Collaborator

Situation

홉 상한 도달의 일시 실패 분류

  • renderer 응답에 hop_cap_hit 필드 추가. 브라우저의 홉 상한 10 소진 표시
  • extractor 의 해당 필드 읽기 없음
    • 결과: 홉 전부 3xx 인 경우 HTML 없음 판정, HEADLESS_UPSTREAM 일시 실패로 종결
    • 결과: dev 실측 렌더 2회. 등록 URL https://httpbingo.org/redirect/12

Task

낭비 재시도 제거

  • 홉 상한 도달의 확정 실패 분류
  • 조건: 계약 카탈로그의 기존 code TOO_MANY_REDIRECTS 사용

Action

홉 상한 도달 판정

  • hop_cap_hit true 이고 쓸 HTML 보유 홉 0건이면 확정 실패
  • 쓸 HTML 보유 홉 1건 이상이면 기존대로 홉 목록 반환
    • 원인: 상한 도달 시에도 앞 홉의 상품 존재 가능
  • 구현: HeadlessRenderResponse.hopCapHit · HeadlessRenderException.hopCapHit() · HttpHeadlessRenderer.renderVerified

구버전 renderer 호환

  • hop_cap_hit 없는 응답은 null 수용. 기존 경로 유지

카탈로그 대조 테스트 등록 제외

  • 원인: 카탈로그의 escalatable 축은 정적 fetch 경로 전용
  • 결과: TOO_MANY_REDIRECTS 의 permanent 대조는 PageFetchException.tooManyRedirects 로 이미 성립

검증

  • 케이스 2건 추가. 홉 전부 302 면 확정 실패, 홉 하나라도 body·dom 보유 시 목록 반환

Result

재시도 예산 절감

  • 홉 상한 도달 건의 렌더 1회로 감소
    • 이전: 렌더 2회 후 확정 실패

연관 이슈

- 왜: 일시 실패(HEADLESS_UPSTREAM)로 나가 core 가 재시도를 낭비했다, dev 실측 렌더 2회
@m-a-king m-a-king added the fix 외부 가시적 결함 수정 label Sep 10, 2026
@m-a-king m-a-king self-assigned this Sep 10, 2026
@github-actions

Copy link
Copy Markdown

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

@coderabbitai

coderabbitai Bot commented Sep 10, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

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

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 0dacc3fd-e5d9-4321-a346-ae551550cab5


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

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

@m-a-king m-a-king changed the title 홉 상한 도달 렌더를 확정 실패로 닫음 홉 상한 도달 렌더 확정 실패 전환 Sep 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix 외부 가시적 결함 수정

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant