Skip to content

非ストリーミング通信のRequest/Responseの分割表示に対応#229

Open
taka2233 wants to merge 30 commits intoDeNA:masterfrom
taka2233:feat/split-history-view
Open

非ストリーミング通信のRequest/Responseの分割表示に対応#229
taka2233 wants to merge 30 commits intoDeNA:masterfrom
taka2233:feat/split-history-view

Conversation

@taka2233
Copy link
Contributor

@taka2233 taka2233 commented Feb 20, 2026

gRPC Streamingなどをストリーミング通信以外のRequestとResponseが1対1で対応づく通信を、Historyタブでマージして表示するようにしました。RequestとResponseが1画面で確認できるため、視認性が向上しています。

以下の条件を全て満たす場合に非ストリーミング通信と判断しています

  • 同一GroupIDでパケットが2つ以下
  • 同一GroupIDでRequestが1つのみ存在
    • gRPC Streamingエンコーダーを使用すると、HEADRSフレームとDATAフレームがRequestとして送信されるため

ストリーミング通信
image

非ストリーミング通信
image

@taka2233 taka2233 force-pushed the feat/split-history-view branch from 9d56758 to 39d1921 Compare February 20, 2026 06:07
@taka2233 taka2233 changed the title Feat/split history view 非ストリーミング通信のRequest/Responseの分割表示に対応 Feb 20, 2026
takatsugu-nakayama and others added 26 commits February 24, 2026 19:20
マージしない場合は、RequestとResponse を分割表示しない。
…and response merging logic; replace hardcoded indices with constants for better readability
@taka2233 taka2233 force-pushed the feat/split-history-view branch from 77a9aaa to 91d6f5a Compare February 24, 2026 10:56
@taka2233 taka2233 marked this pull request as ready for review February 24, 2026 10:57
@funa-tk
Copy link
Contributor

funa-tk commented Mar 1, 2026

https://dena.com/ の通信をとってみたところ、RequestとResponseを一画面で表示できてました。
とても便利です。ストリーミング通信の仕様も問題なさそうです。ありがとうございます。

ただし、下記3点が気になったので修正してもらえると助かります!

  1. HistoryタブのRequestとResponse画面のデフォルト表示がReceived Packetタブになっているで、パッと見、文字化けしているようにみえてしまいまいます。デフォルトでDecodedタブにしてもらえると。
スクリーンショット 2026-03-01 23 57 37
  1. この状態で、最下部のsendボタンを押下しても正常にパケットが送信できてなさそうです。
  2. この状態で、copy BodyをクリックしてもBodyが取得できていなさそうです。

ご確認をお願いいたします!

@taka2233 taka2233 force-pushed the feat/split-history-view branch from 1566694 to dfa9c28 Compare March 4, 2026 13:44
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.

2 participants