Implémenter une meilleure gestion des erreurs #17

New issue

Closed

opened 2021-06-12 00:58:06 +02:00 by l4p1n · 2 comments

l4p1n commented

2021-06-12 00:58:06 +02:00

(Migrated from gitlab.l4p1n.ch)

Pour l'instant, le repository a des unwrap() de partout, ce qui n'est pas forcément idéal.

Le but est, du moins pour les repository, retourner des Result<T, tokio_postgres::Error> ou des eyre::Result<T> ou autre, qui pourront être remontées.

Ensuite, les erreurs qui en sortent devraient pouvoir remonter la chaîne d'appels proprement et être gérées de manière centrale. Je verrai ce qu'il en est pour les erreurs plus spécifiques telles que l'absence de configuration (Option<data_structs::bot::Configuration>), comme ça on pourra aussi le gérer de manière centrale.

L'idéal serait de pouvoir gérer tout ça avec eyre::Result qui offre un moyen de mettre du contexte.

Pour l'instant, le repository a des `unwrap()` de partout, ce qui n'est pas forcément idéal. Le but est, du moins pour les repository, retourner des `Result<T, tokio_postgres::Error>` ou des `eyre::Result<T>` ou autre, qui pourront être remontées. Ensuite, les erreurs qui en sortent devraient pouvoir remonter la chaîne d'appels proprement et être gérées de manière centrale. Je verrai ce qu'il en est pour les erreurs plus spécifiques telles que l'absence de configuration (`Option<data_structs::bot::Configuration>`), comme ça on pourra aussi le gérer de manière centrale. L'idéal serait de pouvoir gérer tout ça avec eyre::Result qui offre un moyen de mettre du contexte.

l4p1n commented

2021-07-01 11:01:42 +02:00

(Migrated from gitlab.l4p1n.ch)

changed title from {-M-}eilleure gestion d{-'-}erreurs{- ?-} to {+Implémenter une m+}eilleure gestion d{+es +}erreurs

changed title from **{-M-}eilleure gestion d{-'-}erreurs{- ?-}** to **{+Implémenter une m+}eilleure gestion d{+es +}erreurs**

l4p1n commented

2021-07-01 11:01:42 +02:00

(Migrated from gitlab.l4p1n.ch)

changed the description