Panic less and with better messages #281

Sloučený

Plume_migration_agent sloučil 2 commity z větve panic-refactor do větve master před

trinity-1686a okomentoval

Vlastník

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 (Migrováno z github.com) schválil tyto změny

elegaanz (Migrováno z github.com) zanechal komentář

Thanks a lot!

elegaanz okomentoval (Migrováno z github.com)

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

trinity-1686a okomentoval

Autor

Vlastník

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 okomentoval (Migrováno z 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 okomentoval

Autor

Vlastník

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

Posuzovatelé

Požadavek na natažení byl sloučen jako 879fde81ba.

Můžete také zobrazit instrukce příkazové řádky.

Krok 1:

Z vašeho repositáře projektu se podívejte na novou větev a vyzkoušejte změny.

git checkout -b panic-refactor master

git pull origin panic-refactor

Krok 2:

Slučte změny a aktualizujte je na Forgejo.

git checkout master

git merge --no-ff panic-refactor

git push origin master

Přihlaste se pro zapojení do konverzace.

Žádní posuzovatelé

Bez milníku

Bez zpracovatelů

2 účastníků

Oznámení

Termín dokončení

Termín dokončení není platný nebo je mimo rozsah. Použijte prosím formát „rrrr-mm-dd“.

Žádný termín dokončení.

Závislosti

Nejsou nastaveny žádné závislosti.

Načítá se…

Není zde žádný obsah.

Štítky Milníky

Panic less and with better messages #281

Posuzovatelé

Krok 1:

Krok 2: