Skip to content

RPC リクエスト処理にタイムアウトを追加する#176

Merged
t-miya merged 11 commits intodevelopfrom
feature/rpc-timeout
Feb 12, 2026
Merged

RPC リクエスト処理にタイムアウトを追加する#176
t-miya merged 11 commits intodevelopfrom
feature/rpc-timeout

Conversation

@t-miya
Copy link
Contributor

@t-miya t-miya commented Feb 10, 2026

RPC リクエスト後、一定時間レスポンスが返ってこなかった場合はタイムアウトするようにする

  • onRPC を廃止し、SendRPCMessage の引数に渡したコールバックで結果を受け取るようにする
  • リクエスト ID を外から渡せないようにして、内部で採番するようにする
  • リクエストは送信時にリクエストIDをキーとした辞書に格納され、レスポンス待ちリクエストとして管理される
  • レスポンス待ちリクエストは リクエストID で管理するため、レスポンスの Json 文字列から ID だけパースして利用する
  • レスポンスの中身のデータは引き続き生のJson文字列が返されるので利用者側でパースする

@t-miya

This comment was marked as resolved.

@github-actions

This comment was marked as resolved.

@t-miya

This comment was marked as resolved.

@github-actions

This comment was marked as resolved.

@t-miya

This comment was marked as resolved.

@github-actions

This comment was marked as resolved.

@t-miya

This comment was marked as resolved.

@github-actions

This comment was marked as resolved.

@t-miya

This comment was marked as resolved.

@github-actions

This comment was marked as resolved.

@t-miya

This comment was marked as resolved.

@github-actions

This comment was marked as resolved.

@t-miya t-miya requested review from torikizi and zztkm February 10, 2026 09:09
Copy link

@zztkm zztkm left a comment

Choose a reason for hiding this comment

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

LGTM です

Copy link
Contributor

@torikizi torikizi left a comment

Choose a reason for hiding this comment

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

ほか大丈夫そうです。ありがとうございました。

@t-miya t-miya merged commit 6a5b1d4 into develop Feb 12, 2026
12 checks passed
@t-miya t-miya deleted the feature/rpc-timeout branch February 12, 2026 02:58
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.

3 participants