no good reason to use thiserror here

freifunk-saar · Dec 31, 2023 · 5ac433a · 5ac433a
1 parent 4119199
commit 5ac433a
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 12 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/Cargo.toml b/Cargo.toml
@@ -19,7 +19,6 @@ serde_json = "1.0"
 serde_repr = "0.1"
 rmp-serde = "1"
 anyhow = "1.0.31"
-thiserror = "1.0"
 url = { version = "2.2", features = ["serde"] }
 base64 = "0.21"
 hex = "0.4.3"

diff --git a/src/cron.rs b/src/cron.rs
@@ -19,7 +19,6 @@ use std::collections::HashMap;
 use anyhow::{bail, Result};
 use diesel::prelude::*;
 use serde_json::{self, json};
-use thiserror::Error;
 
 use rocket::uri;
 
@@ -30,12 +29,6 @@ use crate::routes;
 use crate::schema::*;
 use crate::util::Ctx;
 
-#[derive(Debug, Error)]
-enum NodeListError {
-    #[error("got unsupported version number {version}")]
-    UnsupportedVersion { version: usize },
-}
-
 mod json {
     use chrono::{DateTime, Utc};
     use serde::Deserialize;
@@ -128,9 +121,10 @@ impl<'r> Ctx<'r> {
             .await?;
 
         if cur_nodes.version != 2 {
-            bail!(NodeListError::UnsupportedVersion {
-                version: cur_nodes.version
-            });
+            bail!(
+                "unsupported hopglass node list version: {}",
+                cur_nodes.version
+            );
         }
 
         // Build node HashMap: map node ID to name and online state