Incorrect / overly strict types in 21.4.5
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 35/100
- Tipo di issue
- Bug
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Ferma
- Stack tecnologico
- javascript, node.js, typescript
Direzione di ricerca
Inizia leggendo le modifiche ai tipi più restrittive referenziate in #4562 e riproduci l’errore TypeScript con l’esempio di request.params validato da Joi. Segui le definizioni dei tipi di request.params e request.query, quindi verifica che il parametro numerico validato segnalato non produca più l’errore da stringa a numero, senza indebolire i controlli dei tipi non correlati.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Runtime
Node.js
Runtime version
24.13.1
Module version
21.4.5
Last module version without issue
21.4.4
Used with
No response
Any other relevant information
No response
What are you trying to achieve or the steps to reproduce?
Take advantage of Joi's type handling to directly reference appropriately typed parameters in request.params and request.query. For example:
function greetUser(id: number) {
assert(typeof id === "number", "ID must be a number");
return `Hello, user with ID ${id}!`;
}
server.route({
method: "GET",
path: "/hello/{id}",
handler: (request, _h) => {
// request.params.id is number, not string, because of Joi validation
return { message: greetUser(request.params.id) };
},
options: {
validate: {
params: Joi.object({
id: Joi.number().required(),
}),
},
},
});
What was the result you got?
Starting in Hapi 21.4.5:
Argument of type 'string' is not assignable to parameter of type 'number'.ts(2345)
What result did you expect?
No errors.
I believe that the stricter types in #4562 are too strict. I appreciate stricter typing (even if it breaks compilation in patch releases - #4563) but this seems too strict - from what I understand, relying on Joi to validate queries and route parameters and coerce / convert their types is a core feature of Hapi and is (I assume) a common practice, but the new types act like it isn't permitted / doesn't happen.
- Lingua principale
- JavaScript
- Stelle
- 14.8k
- Fork
- 1.4k
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di hapijs/hapi
-
The data property of a Boom error gets mangled when the error comes from a validatorForse di nuovo libera Una pull request per questa issue è stata chiusa senza essere unita. Apertabug
Difficoltà 3/5 1-2 giorni Idoneità per principianti 70/100
-
The `abort` event on nodejs is deprecatedForse di nuovo libera @akashbirajdar04 l’ha presa 172 giorni fa e non c’è nessuna pull request aperta. Apertabug
Difficoltà 3/5 1-2 giorni Idoneità per principianti 52/100
-
Error-tracking software doesn't catch hapi application errors if an incoming request gets prematurely closedForse di nuovo libera Una pull request per questa issue è stata chiusa senza essere unita. Apertabug
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
-
bug
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
-
feature
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
Issue simili
-
documentation v2
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
modelcontextprotocol/servers#5170 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
SwiftKickMobile/skai#84 ·
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
Billionmail/BillionMail#389 ·
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 92/100
Uniswap/tokenlists-org#3331 ·
-
beginner-friendly category:finding-titles community enhancement first-timers-only good first issue hacktoberfest help wanted low hanging fruit micro-contribution type: task up-for-grabs
Difficoltà 1/5 1-3 ore Idoneità per principianti 85/100
Berserk-hub150/skillhawk#227 · 1 commento ·
I maintainer di solito rispondono entro 8 giorni