akka/akka-http

Restrict / review `Transfer-Encoding` header rendering

Open

#3,761 opened on 2021年2月15日

GitHub で見る
 (0 comments) (0 reactions) (0 assignees)Scala (598 forks)batch import
1 - triageddiscusshackathonhelp wantedt:coret:model

Repository metrics

Stars
 (1,311 stars)
PR merge metrics
 (平均マージ 1d 10h) (30d で 2 merged PRs)

説明

Since #3754, we only support Transfer-Encoding: chunked headers and no other ones. The reasoning is that akka-http is implementing the "Transfer" part of HTTP. Leaving parsing or providing custom Transfer-Encoding headers open to the user will lead to confusion and bugs.

We should check that:

  • The Transfer-Encoding model could make sure that the latest entry is always chunked (see RFC), though that would change the error reporting for parsing as now implemented with #3754
  • User-provided Transfer-Encoding headers will always be ignored

コントリビューターガイド

調査方針
akka httpの現在のTransfer Encoding処理、特にモデルと解析ロジックを調査します。'chunked'のみが最後のエンコーディングであることを強制し、ユーザー提供のTransfer Encodingヘッダーを無視する方法を決定します。
技術スタック
scala
領域
backendapi
Issue 種別
機能
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
半日
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
新着
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
おおむね明確
前提条件
ScalaHTTPAkka
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
40