pages use const routes

src/api/v1/tests/auth.rs

@@ -125,7 +125,7 @@ async fn auth_works() {
     .await;
     assert_eq!(signout_resp.status(), StatusCode::OK);
     let headers = signout_resp.headers();
-    assert_eq!(headers.get(header::LOCATION).unwrap(), "/login")
+    assert_eq!(headers.get(header::LOCATION).unwrap(), PAGES.auth.login);
 }
 
 #[actix_rt::test]

src/main.rs

@@ -28,9 +28,9 @@ mod api;
 mod data;
 mod docs;
 mod errors;
+mod pages;
 mod settings;
 mod static_assets;
-mod templates;
 #[macro_use]
 mod routes;
 #[cfg(test)]
@@ -40,6 +40,7 @@ mod middleware;
 
 pub use api::v1::ROUTES as V1_API_ROUTES;
 pub use data::Data;
+pub use pages::routes::ROUTES as PAGES;
 pub use settings::Settings;
 use static_assets::FileMap;
 
@@ -91,7 +92,7 @@ async fn main() -> std::io::Result<()> {
             .configure(v1::services)
             .configure(docs::services)
             .configure(static_assets::services)
-            .configure(templates::services)
+            .configure(pages::services)
             .app_data(get_json_err())
     })
     .bind(SETTINGS.server.get_ip())

src/middleware/auth.rs

@@ -19,16 +19,15 @@
 use std::task::{Context, Poll};
 
 use actix_identity::Identity;
-//use actix_identity::{CookieIdentityPolicy, IdentityService};
 use actix_service::{Service, Transform};
 use actix_web::dev::{ServiceRequest, ServiceResponse};
 use actix_web::{http, Error, FromRequest, HttpResponse};
 
 use futures::future::{ok, Either, Ready};
 
-pub struct CheckLogin;
+use crate::PAGES;
 
-const LOGIN_ROUTE: &str = "/login";
+pub struct CheckLogin;
 
 impl<S, B> Transform<S> for CheckLogin
 where
@@ -46,25 +45,6 @@ where
         ok(CheckLoginMiddleware { service })
     }
 }
-//
-//pub fn auto_login(req: &HttpRequest, pl: &mut dev::Payload) -> Option<()> {
-//    dbg!("login");
-//    if let Some(_) = Identity::from_request(req, pl)
-//        .into_inner()
-//        .map(|x| x.identity())
-//        .unwrap()
-//    {
-//        Some(())
-//    } else {
-//        None
-//    }
-//}
-//
-//fn not_auth(path: &str) -> bool {
-//    let paths = ["/login", "/css", "/img", "/js"];
-//    paths.iter().any(|x| path.starts_with(x))
-//}
-
 pub struct CheckLoginMiddleware<S> {
     service: S,
 }
@@ -84,9 +64,6 @@ where
     }
 
     fn call(&mut self, req: ServiceRequest) -> Self::Future {
-        //    if not_auth(req.path()) {
-        //        return Either::Left(self.service.call(req));
-        //    };
         let (r, mut pl) = req.into_parts();
 
         // TODO investigate when the bellow statement will
@@ -97,30 +74,14 @@ where
         {
             let req = ServiceRequest::from_parts(r, pl).ok().unwrap();
             Either::Left(self.service.call(req))
-        //            Some(())
         } else {
             let req = ServiceRequest::from_parts(r, pl).ok().unwrap();
             Either::Right(ok(req.into_response(
                 HttpResponse::Found()
-                    .header(http::header::LOCATION, LOGIN_ROUTE)
+                    .header(http::header::LOCATION, PAGES.auth.login)
                     .finish()
                     .into_body(),
             )))
-
-            //None
-        }
-
-        //        let token = auto_login(&r, &mut pl);
-        //        let req = ServiceRequest::from_parts(r, pl).ok().unwrap();
-        //        if token.is_some() {
-        //            Either::Left(self.service.call(req))
-        //        } else {
-        //            Either::Right(ok(req.into_response(
-        //                HttpResponse::Found()
-        //                    .header(http::header::LOCATION, LOGIN_ROUTE)
-        //                    .finish()
-        //                    .into_body(),
-        //            )))
-        //        }
+        }
     }
 }

src/pages/auth/login.rs

@@ -15,7 +15,7 @@
 * along with this program.  If not, see <https://www.gnu.org/licenses/>.
 */
 
-use actix_web::{get, HttpResponse, Responder};
+use actix_web::{HttpResponse, Responder};
 use sailfish::TemplateOnce;
 
 #[derive(Clone, TemplateOnce)]
@@ -34,7 +34,6 @@ impl<'a> Default for IndexPage<'a> {
     }
 }
 
-#[get("/login")]
 pub async fn login() -> impl Responder {
     let body = IndexPage::default().render_once().unwrap();
     HttpResponse::Ok()

src/pages/auth/mod.rs

@@ -17,3 +17,26 @@
 
 pub mod login;
 pub mod register;
+
+pub fn services(cfg: &mut actix_web::web::ServiceConfig) {
+    use crate::define_resource;
+    use crate::PAGES;
+
+    define_resource!(cfg, PAGES.auth.login, Methods::Get, login::login);
+    define_resource!(cfg, PAGES.auth.join, Methods::Get, register::join);
+}
+
+pub mod routes {
+    pub struct Auth {
+        pub login: &'static str,
+        pub join: &'static str,
+    }
+    impl Auth {
+        pub const fn new() -> Auth {
+            Auth {
+                login: "/login",
+                join: "/join",
+            }
+        }
+    }
+}

src/pages/auth/register.rs

@@ -15,7 +15,7 @@
 * along with this program.  If not, see <https://www.gnu.org/licenses/>.
 */
 
-use actix_web::{get, HttpResponse, Responder};
+use actix_web::{HttpResponse, Responder};
 use sailfish::TemplateOnce;
 
 #[derive(TemplateOnce, Clone)]
@@ -34,7 +34,6 @@ impl<'a> Default for IndexPage<'a> {
     }
 }
 
-#[get("/join")]
 pub async fn join() -> impl Responder {
     let body = IndexPage::default().render_once().unwrap();
     HttpResponse::Ok()

src/pages/mod.rs

@@ -19,12 +19,11 @@ use actix_web::web::ServiceConfig;
 
 mod auth;
 mod panel;
+pub mod routes;
 
 pub fn services(cfg: &mut ServiceConfig) {
-    cfg.service(panel::panel);
-    cfg.service(panel::sitekey::add_sitekey);
-    cfg.service(auth::login::login);
-    cfg.service(auth::register::join);
+    auth::services(cfg);
+    panel::services(cfg);
 }
 
 #[cfg(not(tarpaulin_include))]
@@ -60,7 +59,7 @@ mod tests {
         )
         .await;
 
-        let urls = vec!["/", "/sitekey/add"];
+        let urls = vec![PAGES.home, PAGES.panel.sitekey.add];
 
         for url in urls.iter() {
             let resp =
@@ -85,7 +84,7 @@ mod tests {
     #[actix_rt::test]
     async fn public_pages_tempaltes_work() {
         let mut app = test::init_service(App::new().configure(services)).await;
-        let urls = vec!["/join", "/login"];
+        let urls = vec![PAGES.auth.login, PAGES.auth.join];
 
         for url in urls.iter() {
             let resp =

src/pages/panel/mod.rs

@@ -15,11 +15,9 @@
 * along with this program.  If not, see <https://www.gnu.org/licenses/>.
 */
 
-use actix_web::{get, HttpResponse, Responder};
+use actix_web::{HttpResponse, Responder};
 use sailfish::TemplateOnce;
 
-use crate::CheckLogin;
-
 pub mod sitekey;
 
 #[derive(TemplateOnce, Clone)]
@@ -40,10 +38,34 @@ impl<'a> Default for IndexPage<'a> {
     }
 }
 
-#[get("/", wrap = "CheckLogin")]
-pub async fn panel() -> impl Responder {
+pub fn services(cfg: &mut actix_web::web::ServiceConfig) {
+    use crate::define_resource;
+    use crate::PAGES;
+
+    define_resource!(cfg, PAGES.panel.home, Methods::ProtectGet, panel);
+    sitekey::services(cfg);
+}
+
+async fn panel() -> impl Responder {
     let body = IndexPage::default().render_once().unwrap();
     HttpResponse::Ok()
         .content_type("text/html; charset=utf-8")
         .body(body)
 }
+
+pub mod routes {
+    use super::sitekey::routes::Sitekey;
+    pub struct Panel {
+        pub home: &'static str,
+        pub sitekey: Sitekey,
+    }
+
+    impl Panel {
+        pub const fn new() -> Self {
+            Panel {
+                home: "/",
+                sitekey: Sitekey::new(),
+            }
+        }
+    }
+}

src/pages/panel/sitekey/add.rs

@@ -15,11 +15,9 @@
 * along with this program.  If not, see <https://www.gnu.org/licenses/>.
 */
 
-use actix_web::{get, HttpResponse, Responder};
+use actix_web::{HttpResponse, Responder};
 use sailfish::TemplateOnce;
 
-use crate::CheckLogin;
-
 #[derive(TemplateOnce, Clone)]
 #[template(path = "panel/add-site-key/index.html")]
 pub struct IndexPage<'a> {
@@ -44,7 +42,6 @@ impl<'a> Default for IndexPage<'a> {
     }
 }
 
-#[get("/sitekey/add", wrap = "CheckLogin")]
 pub async fn add_sitekey() -> impl Responder {
     let body = IndexPage::default().render_once().unwrap();
     HttpResponse::Ok()

src/pages/panel/sitekey/mod.rs

@@ -16,4 +16,31 @@
 */
 
 mod add;
-pub use add::add_sitekey;
+
+pub mod routes {
+    pub struct Sitekey {
+        pub list: &'static str,
+        pub add: &'static str,
+    }
+
+    impl Sitekey {
+        pub const fn new() -> Self {
+            Sitekey {
+                list: "/sitekey",
+                add: "/sitekey/add",
+            }
+        }
+    }
+}
+
+pub fn services(cfg: &mut actix_web::web::ServiceConfig) {
+    use crate::define_resource;
+    use crate::PAGES;
+
+    define_resource!(
+        cfg,
+        PAGES.panel.sitekey.add,
+        Methods::ProtectGet,
+        add::add_sitekey
+    );
+}

src/pages/routes.rs

@@ -15,14 +15,24 @@
 * along with this program.  If not, see <https://www.gnu.org/licenses/>.
 */
 
-use sailfish::TemplateOnce;
-//use au
+use super::auth::routes::Auth;
+use super::panel::routes::Panel;
+pub const ROUTES: Routes = Routes::new();
 
-//#[get("/")]
-//pub async fn login() -> impl Responder {
-//    let body = SignIn::default().render_once().unwrap();
-//    //        .map_err(|_| ServiceError::InternalError)?;
-//    HttpResponse::Ok()
-//        .content_type("text/html; charset=utf-8")
-//        .body(body)
-//}
+pub struct Routes {
+    pub home: &'static str,
+    pub auth: Auth,
+    pub panel: Panel,
+}
+
+impl Routes {
+    const fn new() -> Routes {
+        let panel = Panel::new();
+        let home = panel.home;
+        Routes {
+            auth: Auth::new(),
+            panel,
+            home,
+        }
+    }
+}

templates/auth/login/index.html

@@ -37,7 +37,7 @@
   <div class="form__secondary-action">
     <p class="form__secondary-action__banner">
       New to mCaptcha?
-      <a href="/join" class="form__secondary-action__link"
+      <a href="<.= crate::PAGES.auth.join .>" class="form__secondary-action__link"
         >Create account</a
       >
     </p>

templates/auth/register/index.html

@@ -57,7 +57,7 @@
   <div class="form__secondary-action">
     <p class="form__secondary-action__banner">
       Already have an account?
-      <a href="/login" class="form__secondary-action__link">Log in</a>
+      <a href="<.= crate::PAGES.auth.login .>" class="form__secondary-action__link">Log in</a>
     </p>
   </div>
 </div>

templates/panel/header/sidebar/index.html

@@ -11,7 +11,7 @@
           <li class="secondary-menu__section-partition"></li>
           -->
     <li class="secondary-menu__item">
-      <a class="secondary-menu__item-link" href="/panel">
+        <a class="secondary-menu__item-link" href="<.= crate::PAGES.home .>">
         <img class="secondary-menu__icon" src="<.= crate::FILES.get("./static-assets/img/svg/home.svg").unwrap() .>" alt="" />
         <div class="secondary-menu__item-name">
           Overview
@@ -19,7 +19,7 @@
       </a>
     </li>
     <li class="secondary-menu__item">
-      <a class="secondary-menu__item-link" href="/panel/sitekey">
+        <a class="secondary-menu__item-link" href="<.= crate::PAGES.panel.sitekey.list .>">
         <img class="secondary-menu__icon" src="<.= crate::FILES.get("./static-assets/img/svg/key.svg").unwrap() .>" alt="" />
         <div class="secondary-menu__item-name">
           Site Keys

templates/panel/taskbar/index.html

@@ -4,7 +4,7 @@
         -->
   <li class="task-bar__spacer"></li>
   <li class="task-bar__action">
-      <a class="task-bar__link" href="/sitekey/add">
+      <a class="task-bar__link" href="<.= crate::PAGES.panel.sitekey.add .>">
         <button class="main-menu__add-site">
             + New Site
         </button>
@@ -22,7 +22,7 @@
   </li>
 
   <li class="task-bar__action">
-      <a href="/logout">
+      <a href="<.= crate::V1_API_ROUTES.auth.logout .>">
     <img class="task-bar__icon" src="<.=
     crate::FILES.get("./static-assets/img/svg/log-out.svg").unwrap() .>" alt="Profile"
     /></a>