abolish http module containg one impl

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

diff --git a/src/api.rs b/src/api.rs
index 90d9a5d916050cf8a0bfa6e5b4e143521e6a69d0..cff02c05e63efd8b40f8b63d4dc7c7e697647421 100644 (file)
--- a/src/api.rs
+++ b/src/api.rs
@@ -3,7 +3,6 @@
 // There is NO WARRANTY.
 
 use crate::imports::*;
-use crate::http::*;
 
 #[derive(Debug,Serialize,Deserialize)]
 struct ApiPiece<O : ApiPieceOp> {
@@ -65,6 +64,27 @@ impl Lens for TransparentLens {
   fn massage_prep_piecestate(&self, _ns : &mut PreparedPieceState) { }
 }
 
+impl<'r> Responder<'r> for OnlineError {
+  #[throws(Status)]
+  fn respond_to(self, req: &Request) -> Response<'r> {
+    let msg = format!("Online-layer error\n{:?}\n{}\n", self, self);
+    use rocket::http::Status;
+    use OnlineError::*;
+    let status = match self {
+      ServerFailure(_) => Status::InternalServerError,
+      NoClient | NoPlayer | GameBeingDestroyed
+        => Status::NotFound,
+      OnlineError::PieceHeld | OnlineError::PieceGone
+        => Status::Conflict,
+      InvalidZCoord | BadJSON(_)
+        => Status::BadRequest,
+    };
+    let mut resp = Responder::respond_to(msg,req).unwrap();
+    resp.set_status(status);
+    resp
+  }
+}
+
 #[throws(OE)]
 fn api_piece_op<O: ApiPieceOp>(form : Json<ApiPiece<O>>)
                    -> impl response::Responder<'static> {

diff --git a/src/http.rs b/src/http.rs
deleted file mode 100644 (file)
index 04b696b..0000000
--- a/src/http.rs
+++ /dev/null
@@ -1,32 +0,0 @@
-// Copyright 2020 Ian Jackson
-// SPDX-License-Identifier: AGPL-3.0-or-later
-// There is NO WARRANTY.
-
-pub use rocket::request::Request;
-pub use rocket::response::{Response,Responder};
-pub use rocket::{post,get,routes};
-pub use rocket_contrib::json::Json;
-pub use rocket::http::Status;
-
-use crate::imports::*;
-
-impl<'r> Responder<'r> for OnlineError {
-  #[throws(Status)]
-  fn respond_to(self, req: &Request) -> Response<'r> {
-    let msg = format!("Online-layer error\n{:?}\n{}\n", self, self);
-    use rocket::http::Status;
-    use OnlineError::*;
-    let status = match self {
-      ServerFailure(_) => Status::InternalServerError,
-      NoClient | NoPlayer | GameBeingDestroyed
-        => Status::NotFound,
-      OnlineError::PieceHeld | OnlineError::PieceGone
-        => Status::Conflict,
-      InvalidZCoord | BadJSON(_)
-        => Status::BadRequest,
-    };
-    let mut resp = Responder::respond_to(msg,req).unwrap();
-    resp.set_status(status);
-    resp
-  }
-}

diff --git a/src/imports.rs b/src/imports.rs
index cdcefcd1de0c9ca2dac0d3dfdc0a63a3ba21b36a..407c0489929aa1eb524d9514a9ae2b1540631752 100644 (file)
--- a/src/imports.rs
+++ b/src/imports.rs
@@ -54,6 +54,12 @@ pub use rocket::request::{FromParam,FromRequest,FromFormValue,LenientForm};
 pub use rocket::response::NamedFile;
 pub use rocket::response;
 
+pub use rocket::request::Request;
+pub use rocket::response::{Response,Responder};
+pub use rocket::{post,get,routes};
+pub use rocket_contrib::json::Json;
+pub use rocket::http::Status;
+
 pub use rand::thread_rng;
 pub use rand::Rng;
 pub use rand::distributions::Alphanumeric;

diff --git a/src/lib.rs b/src/lib.rs
index dd52a1a4277e453e8fbc312afb8f2dc9e283adc9..a08b577a8cb4c9d697fe84a22fddb7232af56207 100644 (file)
--- a/src/lib.rs
+++ b/src/lib.rs
@@ -15,7 +15,6 @@ pub mod keydata;
 pub mod updates;
 pub mod sse;
 pub mod error;
-pub mod http;
 pub mod session;
 pub mod api;
 pub mod spec;

diff --git a/src/session.rs b/src/session.rs
index 6e0c81298db91722df974d58dcc4cca673127a44..5fd918cc5dfa3601ba6ff54f39d8ea7bf13cb963 100644 (file)
--- a/src/session.rs
+++ b/src/session.rs
@@ -3,7 +3,6 @@
 // There is NO WARRANTY.
 
 use crate::imports::*;
-use crate::http::*;
 
 #[derive(Serialize,Debug)]
 struct SessionRenderContext {