metosin/reitit

Option to disable only output validation, not all validation

Open

#602 aperta il 24 mar 2023

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)Clojure (261 fork)batch import
enhancementhelp wanted

Metriche repository

Star
 (1569 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

If I go to reitit.coercion.malli and create my coercion middleware then I can pass :validate option which would enable the validation on input and output of the endpoints. But if I'd want to only disable it for output and still include it in the Swagger spec I can't do it.

I think it would be a good idea to have the :validate option to not use a boolean but instead have it as a map:

:validate {:encode true, :decode true}

What do you think?

It would also be good to expose it as a map from calling create function and not hacking through by copying the code.

Guida contributor

Direzione di ricerca
Rivedere l'implementazione corrente dell'opzione `:validate` in `reitit.coercion.malli` e della funzione `create`. Comprendere come la validazione viene applicata a input e output. Considerare di modificare l'opzione per accettare una mappa con chiavi `:encode` e `:decode`, garantendo la compatibilità con le versioni precedenti. Verificare la coerenza dei test e della documentazione esistenti.
Tech stack
Nessuno
Dominio
backend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Clojure
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60