Multipart Form Data "RoutingContext::fileUploads"
まだ誰も着手していません。
評価
- 難易度
- 5/5
- 見積もり時間
- 1週間以上
- 初心者へのやさしさ
- 35/100
調査の方向性
まず、RoutingContext::fileUploads、RoutingContext の body、body handler、および検証済みデータの map 周辺の multipart 処理を調査します。ファイルアップロード、空でない Content-Disposition のファイル名、ディスクへのストリーミング、body handler のアップロードサイズ制限を対象とするテストを確認または追加します。過剰なメモリ使用なしに、これらの値と動作を利用できるようになれば完了です。
索引モデルが issue の本文から書いたものです。
説明
Describe the feature
Multipart form data has changed quite a bit with this new strategy. I'm not sure if this is the actual plan going forward, but we're losing what I think is pretty crucial functionality from what I can tell. One of which being that files used to be streamed to disk and then available to be streamed somewhere else from the file. The other is the filename of the file uploaded in the multipart form data, which comes from the content disposition header.
Maybe this already exists through some other form, but I don't see any tests in this repo that upload file, set a nonempty content disposition or use the body handler for limiting file upload size.
Use cases
Streaming files means that there's less risk of out of memory exceptions. Also, the content disposition header is standard and being unable to access its value through some means is weird.
These are probably just not supported yet, but I find it pretty weird how many breaking changes are being introduced. RoutingContext::fileUploads and the RoutingContext body just aren't used anymore from what I can tell. The validated body has to be pulled from the data map now in the context instead.
Contribution
Who should implement this feature ? are you volunteering for implementing this feature or
do you know that is able and willing implement this feature ?
- 主要言語
- Java
- スター
- 15
- フォーク
- 19
- PR マージ指標
- 30日以内にマージされた PR はありません
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
eclipse-vertx/vertx-openapi のほかの issue
-
bug
難易度 3/5 1〜2日 初心者へのやさしさ 55/100
eclipse-vertx/vertx-openapi#137 · コメント 5 件 ·
-
enhancement help wanted
難易度 4/5 3〜5日 初心者へのやさしさ 45/100
eclipse-vertx/vertx-openapi#136 · コメント 1 件 ·
-
enhancement
難易度 3/5 1〜2日 初心者へのやさしさ 38/100
eclipse-vertx/vertx-openapi#122 · コメント 4 件 ·
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 48/100
eclipse-vertx/vertx-openapi#117 · コメント 6 件 ·
-
enhancement
難易度 5/5 1週間以上 初心者へのやさしさ 35/100
eclipse-vertx/vertx-openapi#93 · コメント 1 件 ·
eclipse-vertx/vertx-openapi の issue をすべて見る
似ている issue
-
難易度 1/5 1時間未満 初心者へのやさしさ 75/100
521xueweihan/HelloGitHub#4105 ·
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
メンテナーはふだん 1 日以内に返信
-
code-maintenance
難易度 2/5 1〜3時間 初心者へのやさしさ 80/100
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1時間未満 初心者へのやさしさ 85/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
krizaka/krizaka-notifications#32 ·
メンテナーはふだん 1 日以内に返信