cargo check: Fix last warning One Result return value went unchecked, and that could lead to untrackable bugs. This has now been checked, and prints a log line.

zj

Sep 21, 2021, 9:35 AM

ZMJZQ2NOAQJLC6FMRVC6P2I22XYHHUKAPC7HQFIGDQVYPEC5SUKAC

Dependencies

• [2] S6TFYMRG routes: Move pijul routes to controllers module As with the root routes, now the pijul routes are moved. The mounting of the routes is still done in main.rs though the controller module now collects them. This should DRY this code
• [3] GT4TOOS4 cargo check: Fix identities route complaints As the route is at this time not doing anything as identities cannot be stored right now, the endpoint can ignore a lot of path elements. This fixes `cargo check` for the most part.

Change contents

• replacement in src/controllers/pijul.rs at line 85
  [2.2529]→[2.2529:2578](∅→∅)
  − repo.changestore().ensure_parent_dirs(hash);
  [2.2529]

  [2.2578]
  +

  + if repo.changestore().ensure_parent_dirs(hash).is_err() {

  + println!("No write permission?");

  + return rocket::http::Status::InternalServerError;

  + }
• edit in src/controllers/pijul.rs at line 96
  [2.2732]→[2.2732:2757](∅→∅)
  − // TODO clean up