withValidator({ query }) keeps only the last value of a repeated query key
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 78/100
- Type d'issue
- Bug
- Clarté
- Clairement spécifiée
- Activité
- Active
- Stack technique
- typescript
- Domaine
- backend-api-design
Piste de recherche
Commencez par runValidation dans src/runtime/validator.ts, où l’issue indique que les paramètres de requête sont copiés une clé à la fois. Vérifiez comment c.req.queries() représente les clés répétées et comparez ce comportement à celui du validateur de requête de Hono. Le travail est terminé lorsqu’une clé répétée atteint le schéma sous forme de tableau, qu’une clé unique reste une chaîne et que des tests pertinents couvrent le comportement existant de validation des requêtes.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
What happens
When a query key appears more than once, withValidator({ query }) passes only its last value to the schema. The earlier values are dropped without any error.
export const loader = defineHandler(
withValidator({ query: z.object({ tag: z.union([z.string(), z.array(z.string())]) }) }),
(c) => c.input.query,
);
GET /notes?tag=work&tag=urgent
- Expected:
{ tag: ['work', 'urgent'] } - Actual:
{ tag: 'urgent' }
A schema that expects an array, like z.object({ tag: z.array(z.string()) }), never passes: Void always gives it a string.
Cause
runValidation in src/runtime/validator.ts copies the search params one key at a time, so each repeated key overwrites the one before:
const raw = {};
for (const [key, value] of url.searchParams) raw[key] = value;
Suggested fix
Do what Hono's validator('query') does: a key that appears once stays a string, and a key that repeats becomes an array.
const raw = {};
for (const [key, values] of Object.entries(c.req.queries())) {
raw[key] = values.length === 1 ? values[0] : values;
}
For keys that appear once, this works exactly as before, so existing schemas aren't affected.
Why it matters
Multi-select filters (?tag=a&tag=b) are a common pattern, and HTML forms send a <select multiple> or a group of checkboxes this way. Other frameworks keep every value: Hono's own validator and Nuxt's getQuery() return arrays, and React Router and SvelteKit have searchParams.getAll().
Version: void 1.4.0
- Langage dominant
- TypeScript
- Étoiles
- 580
- Forks
- 56
- Merge moyen
- 12 h 10 min
- PR mergées (30 j)
- 163
Préparer son environnement
Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
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 voidzero-dev/void
-
void deploy cannot deploy a Worker with no URL after its first deploy, and --atomic refuses itOuverte
Difficulté 4/5 1-2 jours Accessibilité débutants 48/100
voidzero-dev/void#644 ·
Les mainteneurs répondent en général sous 1 jour
-
React: `action()` and `router.visit()` never resolve when awaited in a transitionPeut-être pris @cpojer l’a pris aujourd’hui. Ouverte
voidzero-dev/void#643 · 1 personne assignée ·
Les mainteneurs répondent en général sous 1 jour
-
Static output always writes `foo.html`, so trailing-slash URLs 404 on hosts like GitHub PagesOuverte
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 42/100
voidzero-dev/void#638 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 3/5 1-2 jours Accessibilité débutants 57/100
voidzero-dev/void#636 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 3/5 1-2 jours Accessibilité débutants 67/100
voidzero-dev/void#635 ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de voidzero-dev/void
Issues similaires
-
Replace discontinued crypto-js in the themePeut-être pris @aelmanaa l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
PaloAltoNetworks/docusaurus-openapi-docs#1689 ·
Les mainteneurs répondent en général sous 2 jours
-
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
solana-foundation/solana-web3.js#4015 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
formatSearchQuery drops the :present modifier, so `email:present=true` is sent as `email=true`Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
Les mainteneurs répondent en général sous 1 jour
-
documentation
Difficulté 1/5 1-3 heures Accessibilité débutants 90/100
ivopogace/opencode-courier#422 ·
Les mainteneurs répondent en général sous 1 jour
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
chapel-lang/chapel-vscode#158 ·
Les mainteneurs répondent en général sous 1 jour