[Bug] Land returns codes.Unknown instead of InvalidArgument for an unrecognized strategy
メンテナーはふだん 2 日以内に返信
評価
- 難易度
- 1/5
- 見積もり時間
- 1時間未満
- 初心者へのやさしさ
- 22/100
調査の方向性
Start in service/submitqueue/gateway/server/mapper/land.go, where ProtoToLandRequest and resolveLandStrategy return a plain fmt.Errorf for an unknown strategy. Then check gatewayStatusError to see how it classifies errors via controller.IsInvalidRequest. Done means the mapper error is wrapped as a user error so status.Code(err) returns InvalidArgument. Pull request #997 is already open against this, so check it before starting.
索引モデルが issue の本文から書いたものです。
説明
Description
SubmitQueueGateway.Land called with an unrecognized Strategy enum value (for example one sent by a newer client) is answered with gRPC Unknown instead of InvalidArgument. mapper.ProtoToLandRequest returns a plain fmt.Errorf, and gatewayStatusError passes unclassified errors through unchanged, so gRPC reports Unknown. Every other invalid-input error from Land is classified as a user error and surfaces as InvalidArgument.
Steps to Reproduce
- Start the local stack (
make local-submitqueue-start). - Call
SubmitQueueGateway.Landwith a configured queue, a valid change URI, andStrategy: mergestrategypb.Strategy(9999). - Inspect
status.Code(err).
Expected Behavior
codes.InvalidArgument, consistent with the gateway's other invalid-input errors.
Actual Behavior
codes.Unknown, with the message failed to map land strategy: unknown land strategy in proto message: 9999.
Environment
- Go version: go1.27.1
- OS: Windows
- Bazel version: 8.4.1
Additional Context
resolveLandStrategy and ProtoToLandRequest are in service/submitqueue/gateway/server/mapper/land.go. errs.NewUserError already matches controller.IsInvalidRequest in gatewayStatusError, so wrapping the mapper error is enough. I have a fix ready.
- 主要言語
- Go
- スター
- 247
- フォーク
- 16
- 平均マージ
- 2日 1時間
- マージ済み PR（30日）
- 125
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
uber/submitqueue のほかの issue
-
Batch-scoped DLQ reconciler fails requests that another live batch owns対応中かも @burgulvedant が 14 日前に担当しました。 オープンbug
難易度 4/5 3〜5日 初心者へのやさしさ 55/100
uber/submitqueue#826 · コメント 1 件 ·
メンテナーはふだん 2 日以内に返信
-
Cancel misses a batch mid-promotion, letting a cancelled request reach speculation再び着手できるかも @burgulvedant が 14 日前に担当しましたが、オープン中のプルリクエストはありません。 オープンbug
難易度 4/5 3〜5日 初心者へのやさしさ 45/100
uber/submitqueue#825 · コメント 1 件 ·
メンテナーはふだん 2 日以内に返信
-
bug
難易度 4/5 3〜5日 初心者へのやさしさ 52/100
uber/submitqueue#824 · コメント 2 件 ·
メンテナーはふだん 2 日以内に返信
-
bug
難易度 4/5 3〜5日 初心者へのやさしさ 55/100
uber/submitqueue#820 ·
メンテナーはふだん 2 日以内に返信
-
bug
難易度 5/5 1週間以上 初心者へのやさしさ 35/100
uber/submitqueue#819 · コメント 1 件 ·
メンテナーはふだん 2 日以内に返信
uber/submitqueue の issue をすべて見る
似ている issue
-
難易度 1/5 1時間未満 初心者へのやさしさ 72/100
kubescape/node-agent#1099 ·
メンテナーはふだん 1 日以内に返信
-
import-export
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
genealogix/glx#1498 ·
メンテナーはふだん 2 日以内に返信
-
Docs: 2 broken linksオープンBug pulumi/pulumi-terraform-bridge R2 S2
難易度 1/5 1時間未満 初心者へのやさしさ 72/100
pulumi/pulumi-terraform-bridge#3713 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
ossf/pvtr-github-repo-scanner#539 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
goreleaser/goreleaser#7316 ·
メンテナーはふだん 1 日以内に返信