Panic less and with better messages #281

Fusionnée

Plume_migration_agent a fusionné 2 révision(s) à partir de panic-refactor vers master

trinity-1686a a commenté

Propriétaire

Fix #21
Change most unwrap to expect, and don't panic when it can be avoided. Also return better http codes, i.e. 400 and 404, when appropriate.
The trait rocker::request::FromParam should probably be implemented for Post, Blog, Media, User and Tag. Then we wouldn't need to return Options whenever an entity does not exist, and just let Rocket do the work for us

Fix #21 Change most unwrap to expect, and don't panic when it can be avoided. Also return better http codes, i.e. 400 and 404, when appropriate. The trait `rocker::request::FromParam` should probably be implemented for Post, Blog, Media, User and Tag. Then we wouldn't need to return `Option`s whenever an entity does not exist, and just let Rocket do the work for us

👍 1 ❤️ 1

elegaanz (Migré de github.com) ces changements ont été approuvés

elegaanz (Migré de github.com) laisser un commentaire

Thanks a lot!

elegaanz a commenté (Migré de github.com)

And yes, implementing FromParams for these types would be a good idea.

trinity-1686a a commenté

Éditeur

Propriétaire

After trying to, I guess it might not be possible, because we need a db connection, and I don't think there is a way to access managed state from from_param (it only takes a &RawStr as parameters). Nonetheless I'll ask on Matrix

elegaanz a commenté (Migré de github.com)

Maybe with with FromRequest and Request::get_param then? But it may force us to duplicate parameters in the routes signatures…

Maybe with with `FromRequest` and `Request::get_param` then? But it may force us to duplicate parameters in the routes signatures…

trinity-1686a a commenté

Éditeur

Propriétaire

If we end up doing manual routing it may reduce code duplication a bit, but it will add a lot of complexity, and we won't be able to use rocket's code generation if we do so. I'll see what answer I get from Matrix, but what I said earlier might very well be a false good idea

Relecteurs

La demande d'ajout a été fusionnée en 879fde81ba.

Vous pouvez également voir les instructions en ligne de commande.

Étape 1:

Depuis le dépôt de votre projet, sélectionnez une nouvelle branche et testez les modifications.

git checkout -b panic-refactor master

git pull origin panic-refactor

Étape 2:

Fusionner les modifications et mettre à jour sur Forgejo.

git checkout master

git merge --no-ff panic-refactor

git push origin master

Connectez-vous pour rejoindre cette conversation.

Aucune évaluation

Aucun jalon

Pas d'assignataires

2 participants

Notifications

Échéance

La date d’échéance est invalide ou hors plage. Veuillez utiliser le format 'aaaa-mm-dd'.

Aucune échéance n'a été définie.

Dépendances

No dependencies set.

Chargement…

Il n'existe pas encore de contenu.

Étiquettes Jalons

Panic less and with better messages #281

Relecteurs

Étape 1:

Étape 2: