Skip to content

[#54] HTTP 멱등성과 Retry-After를 반영한 재시도 정책을 구성한다 - #58

Merged
opficdev merged 7 commits into
developfrom
feat/#54
Aug 22, 2026
Merged

opficdev merged 7 commits into
developfrom
feat/#54

Conversation

@opficdev

Copy link
Copy Markdown
Owner

🔗 연관된 이슈

📝 작업 내용

📌 요약

  • HTTP 멱등성과 Retry-After를 반영한 결정적 재시도 정책 구성

🔍 상세

  • NXRetryPolicy에 기본 멱등 method, POST·PATCH 명시 허용, 서버 지연 상한, local jitter 정책 추가
  • NXRetryInterceptor에 method별 재시도 판단, 429·503의 Retry-After 초·HTTP-date 해석, 서버 지연 상한 적용 추가
  • RFC 850 두 자리 연도, GMT 형식, 음수·부호·overflow 값의 fallback 처리 추가
  • 내부 NXRetryExecutionDependencies를 통한 clock·sleeper·무작위 값 주입 경계 구성
  • 취소 뒤 다음 attempt 차단과 requestStart·requestEnd·requestFailure·retry log 순서 회귀 테스트 추가
  • README.md, README.ko.md, DocC에 기본 재시도 method, opt-in 조건, Retry-After, jitter 동작 안내 추가

검증

  • swift build 성공
  • swift test 성공 — 63개 테스트, 12개 suite
  • 변경 Swift 파일 SwiftLint 0건
  • git diff --check 성공

📸 영상 / 이미지 (Optional)

@opficdev opficdev self-assigned this Aug 22, 2026

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: a6bf394bb4

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread Sources/Nexa/Runtime/NXRetryInterceptor.swift
Comment thread Sources/Nexa/Runtime/NXRetryExecutionDependencies.swift Outdated
@opficdev
opficdev merged commit 0c8e7d1 into develop Aug 22, 2026
4 checks passed
@opficdev
opficdev deleted the feat/#54 branch August 22, 2026 16:34
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.

HTTP 멱등성과 Retry-After를 반영한 재시도 정책을 구성한다

1 participant