FileUploader should surface why an upload failed instead of returning a bare bool
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
調査の方向性
Start by locating the FileUploader.upload entry point and tracing how it handles HTTP responses and exceptions. Review the upload behavior introduced by #5722 and the related context in #981; the issue does not identify files or tests. Done means callers can distinguish upload failure reasons while existing bool methods retain their behavior, with the proposed result-versus-exception design resolved.
索引モデルが issue の本文から書いたものです。
説明
Problem to Solve
FileUploader.upload returns Future<bool>. On failure it drops the HTTP status and swallows every exception, so app code cannot tell a file that was too large from an expired upload description or a network error.
Since #5722 the server answers every rejected upload with a distinct status and a reason, and S3 / GCS already return their own 4xx. See #981.
Proposal
Add a method that returns the outcome, e.g. FileUploadResult with success, statusCode, reason (the response body) and error, and keep the existing bool methods as wrappers. Alternatively throw a FileUploadException, but that changes behavior for existing callers.
Use Case
An app uploads a camera photo and wants to tell the user "file too large" and retry smaller, instead of a generic error.
- 主要言語
- Dart
- スター
- 3.3k
- フォーク
- 382
- 平均マージ
- 2日 10時間
- マージ済み PR（30日）
- 55
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
serverpod/serverpod のほかの issue
-
area: templates bug
難易度 1/5 1時間未満 初心者へのやさしさ 72/100
メンテナーはふだん 1 日以内に返信
-
create-repair-migration should print the server address it connects to対応中かも @mackings が 16 日前に担当しました。 オープンenhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 80/100
serverpod/serverpod#5849 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
FallbackMiddleware condition can't inspect the request再び着手できるかも @mackings が 17 日前に担当しましたが、オープン中のプルリクエストはありません。 オープンarea: web server enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
serverpod/serverpod#5829 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
area: configuration area: runtime
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
serverpod/serverpod#3749 · リアクション 1 件 ·
メンテナーはふだん 1 日以内に返信
-
area: cli enhancement stake: onboarding
難易度 2/5 1〜3時間 初心者へのやさしさ 60/100
メンテナーはふだん 1 日以内に返信
serverpod/serverpod の issue をすべて見る
似ている issue
-
@apps/catalog enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
ymm-oss/flutter-mobile-project-template#699 ·
メンテナーはふだん 15 日以内に返信
-
area::build area::tests
難易度 2/5 1〜3時間 初心者へのやさしさ 69/100
linagora/twake-on-matrix#3504 · コメント 1 件 ·
メンテナーはふだん 3 日以内に返信
-
[Packages] [flutter_shaders] InkWell shader golden test fails on Flutter stable and Master channelsオープンfrom: manual-qa
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
メンテナーはふだん 1 日以内に返信
-
README highlightsオープンdocumentation
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
patarapolw/lingua_srs#22 ·
-
interruptRestart fails with "Tool <name> not found" for namespaced tools (ns/tool)対応中かも @esrakadah が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
genkit-ai/genkit-dart#657 ·
メンテナーはふだん 1 日以内に返信