Skip to content

[Fix]: OkHttp 엔진 타임아웃 명시적 설정 - #160

Open
oungsi2000 wants to merge 1 commit into
developfrom
fix/signup-timout
Open

[Fix]: OkHttp 엔진 타임아웃 명시적 설정#160
oungsi2000 wants to merge 1 commit into
developfrom
fix/signup-timout

Conversation

@oungsi2000

Copy link
Copy Markdown
Collaborator

관련 이슈

작업 내용

회원가입 시 Ktor HttpTimeout 설정이 반영되지 않는 문제 수정

변경사항 / 상세

  • Ktor HttpTimeout 플러그인만으로는 OkHttp 엔진 기본 소켓 타임아웃(10s)이 우선 적용되어 설정한 100s 타임아웃이 무시되는 문제가 있었음
  • NetworkModule의 OkHttp 엔진(engine { config { ... } })에도 동일한 100s 타임아웃(connect/read/write)을 명시적으로 지정

중점 리뷰사항

  • 타임아웃 값(100s)이 기존 HttpTimeout 설정값과 일치하는지

스크린샷 (선택)

🤖 Generated with Claude Code

@coderabbitai

coderabbitai Bot commented Aug 24, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: a5d51714-b6dd-4320-99cc-245a52c7dc69

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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.

@oungsi2000 oungsi2000 changed the title fix: OkHttp 엔진 타임아웃 명시적 설정 [Fix]: OkHttp 엔진 타임아웃 명시적 설정 Aug 24, 2026
@oungsi2000 oungsi2000 self-assigned this Aug 24, 2026
@oungsi2000 oungsi2000 added the fix 버그 수정 label Aug 24, 2026
Ktor HttpTimeout 플러그인만으로는 OkHttp 엔진 기본 소켓 타임아웃(10s)이
우선 적용되어 설정이 반영되지 않는 문제가 있어, 엔진 레벨에도 동일한
타임아웃(100s)을 직접 지정한다.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_015do5t4ThrkmdyfYb4gkbCq
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