HTTPClientRequest looks & feels like a value type but isn't
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 5/5
- 見積もり時間
- 1週間以上
- 初心者へのやさしさ
- 35/100
- issue の種類
- リファクタリング
- 明瞭さ
- おおむね明確
- 活発さ
- 停滞
- 技術スタック
- swift
- 領域
- api, networking
調査の方向性
まず HTTPClientRequest と現在の HTTP クライアント実行 API を確認し、次にそれらを提案されている execute(request, body:) 形式と比較します。リクエストとストリーム可能な body を分離し、正しい一度だけの消費動作を維持することで合意された API 設計ができれば完了です。
索引モデルが issue の本文から書いたものです。
説明
Sadly HTTPClientRequest contains a streamable body which is typically a reference to a (frequently) consume-once AsyncSequence. That means it's not actually a value type.
That's a major API winkle that should be fixed as soon as feasible.
My recommendation would be to separate the request and the body. Something like try await httpClient.execute(request, body: myStream).
- 主要言語
- Swift
- スター
- 1.1k
- フォーク
- 156
- 平均マージ
- 1時間 54分
- マージ済み PR（30日）
- 4
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
swift-server/async-http-client のほかの issue
-
RequestBag keeps the request body after an early response with status >= 300対応中かも @beshkenadze が 1 日前に担当しました。 オープン
難易度 3/5 1〜2日 初心者へのやさしさ 68/100
swift-server/async-http-client#941 ·
メンテナーはふだん 1 日以内に返信
-
HTTP/2 stream stays half-closed after an early response with status >= 300対応中かも @beshkenadze が 1 日前に担当しました。 オープン
難易度 4/5 3〜5日 初心者へのやさしさ 55/100
swift-server/async-http-client#940 ·
メンテナーはふだん 1 日以内に返信
-
Transaction keeps the request body after an early response with status >= 300対応中かも @beshkenadze が 1 日前に担当しました。 オープン
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
swift-server/async-http-client#939 ·
メンテナーはふだん 1 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 22/100
swift-server/async-http-client#938 ·
メンテナーはふだん 1 日以内に返信
-
Invalid state: idleオープン
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
swift-server/async-http-client#927 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
swift-server/async-http-client の issue をすべて見る
似ている issue
-
ci-failure ci:assertion test
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
openwatersio/slackwater-ios#811 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 63/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
foxglove/mcap#1907 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
good first issue help wanted translation
難易度 2/5 1〜3時間 初心者へのやさしさ 83/100
-
[Tool] Annotatorオープンhas-image has-license has-readme new-tool ready-to-approve repo-verified
難易度 1/5 1時間未満 初心者へのやさしさ 78/100
shanselman/TinyToolTown#873 ·
メンテナーはふだん 5 日以内に返信