[Bug] Land returns codes.Unknown instead of InvalidArgument for an unrecognized strategy
Les mainteneurs répondent en général sous 2 jours
Évaluation
- Difficulté
- 1/5
- Temps estimé
- Moins d'une heure
- Accessibilité débutants
- 22/100
Piste de recherche
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.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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.
- Langage dominant
- Go
- Étoiles
- 247
- Forks
- 16
- Merge moyen
- 2 j 1 h
- PR mergées (30 j)
- 125
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de uber/submitqueue
-
Batch-scoped DLQ reconciler fails requests that another live batch ownsPeut-être pris @burgulvedant l’a pris il y a 14 jours. Ouvertebug
Difficulté 4/5 3-5 jours Accessibilité débutants 55/100
uber/submitqueue#826 · 1 commentaire ·
Les mainteneurs répondent en général sous 2 jours
-
Cancel misses a batch mid-promotion, letting a cancelled request reach speculationPeut-être à nouveau libre @burgulvedant l’a pris il y a 14 jours, et aucune pull request n’est ouverte. Ouvertebug
Difficulté 4/5 3-5 jours Accessibilité débutants 45/100
uber/submitqueue#825 · 1 commentaire ·
Les mainteneurs répondent en général sous 2 jours
-
bug
Difficulté 4/5 3-5 jours Accessibilité débutants 52/100
uber/submitqueue#824 · 2 commentaires ·
Les mainteneurs répondent en général sous 2 jours
-
bug
Difficulté 4/5 3-5 jours Accessibilité débutants 55/100
uber/submitqueue#820 ·
Les mainteneurs répondent en général sous 2 jours
-
bug
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 35/100
uber/submitqueue#819 · 1 commentaire ·
Les mainteneurs répondent en général sous 2 jours
Toutes les issues de uber/submitqueue
Issues similaires
-
ManageSieve proxy: a backend's NO/BYE greeting is skipped as a capability line and reported as EOFOuvertebug
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
Les mainteneurs répondent en général sous 1 jour
-
Mend: dependency security vulnerability
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
ManageIQ/manageiq-pods#1476 ·
Les mainteneurs répondent en général sous 1 jour
-
mcp: typed tool handlers lose wrapped JSON-RPC errorsPeut-être à nouveau libre Une pull request pour cette issue a été fermée sans être fusionnée. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
modelcontextprotocol/go-sdk#1529 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 72/100
kubescape/node-agent#1099 ·
Les mainteneurs répondent en général sous 1 jour
-
import-export
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
genealogix/glx#1498 ·
Les mainteneurs répondent en général sous 2 jours