finagle/finch

Better errors for missing/wrong JSON properties

Aperta

#264 aperta il 6 mag 2015

Vedi su GitHubGuida contributor
 (3 commenti) (1 reazione) (1 assegnatario)Scala (218 fork)batch import
featurehacktoberfestready

Metriche repository

Star
 (1594 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

It would be nice if we could better understand what's wrong with an incoming JSON body and throw a reasonable error about it. For now, we throw NotParsed(BodyItem, ....). Would be nice if we could promote BodyItem to a case class BodyItem(name: String) and wrap missing/wrongly decoded properties in it.

This is only about the cases when JSON is decoded directly to a case class: body.as[User]. I believe, end users should handle this manually (with custom exceptions like NotFoundJsonProperty) when dealing with raw JSON types directly: body.as[Json] and then parsing them with AST API provided by a JSON library.

See discussion in #261 started by @lderks.

Guida contributor

Direzione di ricerca
Indaga come funziona body.as[User] e identifica dove vengono generati i messaggi di errore. Quindi implementa un errore migliore che includa il nome della proprietà mancante o errata.
Tech stack
scala
Dominio
backend api design
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
Scala
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40