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 il y a 1 jour. 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
-
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
Northeastern-Electric-Racing/FinishLine#4660 ·
Les mainteneurs répondent en général sous 2 jours
-
BeOS .json missingOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
mihaip/infinite-mac#475 ·
-
Let TypeScript accept an initial object in ReactiveDictPeut-être pris @yograj-vijay-yadav l’a pris aujourd’hui. Ouvertebug good first issue help wanted workshop
Difficulté 2/5 1-3 heures Accessibilité débutants 92/100
meteor/meteor#14919 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 83/100
marmelab/atomic-crm#401 ·
Les mainteneurs répondent en général sous 1 jour
-
account: profile
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
Les mainteneurs répondent en général sous 5 jours