Revert ":recycle: refactor(API): provide a minor redesign to the internal api to support parsing of `json` data (#427)"

This reverts commit 729c55068844b2c69600ccb19cdfc9099f93e6c8.

src/models/parser_models.rs

@@ -10,7 +10,7 @@
 /// order to allow the deserializing the json back to struct in aggregate function in
 /// aggregator.rs and create a new struct out of it and then serialize it back to json and pass
 /// it to the template files.
-#[derive(Default, Clone)]
+#[derive(Default)]
 pub struct Style {
     /// It stores the parsed theme option used to set a theme for the website.
     pub theme: String,

src/models/server_models.rs

@@ -1,15 +1,17 @@
 //! This module provides the models to parse cookies and search parameters from the search
 //! engine website.
+use std::borrow::Cow;
+
+use serde::Deserialize;
 
 use super::parser_models::Style;
-use serde::{Deserialize, Serialize};
 
 /// A named struct which deserializes all the user provided search parameters and stores them.
 #[derive(Deserialize)]
 pub struct SearchParams {
     /// It stores the search parameter option `q` (or query in simple words)
     /// of the search url.
-    pub q: Option<String>,
+    pub q: Option<Cow<'static, str>>,
     /// It stores the search parameter `page` (or pageno in simple words)
     /// of the search url.
     pub page: Option<u32>,
@@ -20,29 +22,26 @@ pub struct SearchParams {
 
 /// A named struct which is used to deserialize the cookies fetched from the client side.
 #[allow(dead_code)]
-#[derive(Deserialize, Serialize)]
-pub struct Cookie {
+#[derive(Deserialize)]
+pub struct Cookie<'a> {
     /// It stores the theme name used in the website.
-    pub theme: String,
+    pub theme: Cow<'a, str>,
     /// It stores the colorscheme name used for the website theme.
-    pub colorscheme: String,
-    /// It stores the animation name used for the website theme.
-    pub animation: Option<String>,
+    pub colorscheme: Cow<'a, str>,
     /// It stores the user selected upstream search engines selected from the UI.
-    pub engines: Vec<String>,
+    pub engines: Cow<'a, [Cow<'a, str>]>,
     /// It stores the user selected safe search level from the UI.
     pub safe_search_level: u8,
 }
 
-impl Cookie {
+impl<'a> Cookie<'a> {
     /// server_models::Cookie contructor function
-    pub fn build(style: Style, mut engines: Vec<String>, safe_search_level: u8) -> Self {
+    pub fn build(style: &'a Style, mut engines: Vec<Cow<'a, str>>, safe_search_level: u8) -> Self {
         engines.sort();
         Self {
-            theme: style.theme.clone(),
-            colorscheme: style.colorscheme.clone(),
-            animation: style.animation.clone(),
-            engines,
+            theme: Cow::Borrowed(&style.theme),
+            colorscheme: Cow::Borrowed(&style.colorscheme),
+            engines: Cow::Owned(engines),
             safe_search_level,
         }
     }

src/server/routes/search.rs

@@ -15,7 +15,7 @@ use actix_web::{get, http::header::ContentType, web, HttpRequest, HttpResponse};
 use itertools::Itertools;
 use regex::Regex;
 use std::time::{SystemTime, UNIX_EPOCH};
-use tokio::time::Duration;
+use std::{borrow::Cow, time::Duration};
 use tokio::{
     fs::File,
     io::{AsyncBufReadExt, BufReader},
@@ -54,15 +54,17 @@ pub async fn search(
             let cookie = req.cookie("appCookie");
 
             // Get search settings using the user's cookie or from the server's config
-            let mut search_settings: server_models::Cookie = cookie
+            let mut search_settings: server_models::Cookie<'_> = cookie
                 .and_then(|cookie_value| serde_json::from_str(cookie_value.value()).ok())
                 .unwrap_or_else(|| {
                     server_models::Cookie::build(
-                        config.style.clone(),
+                        &config.style,
                         config
                             .upstream_search_engines
                             .iter()
-                            .filter_map(|(engine, enabled)| enabled.then_some(engine.clone()))
+                            .filter_map(|(engine, enabled)| {
+                                enabled.then_some(Cow::Borrowed(engine.as_str()))
+                            })
                             .collect(),
                         config.safe_search,
                     )
@@ -159,7 +161,7 @@ async fn results(
     cache: &'static SharedCache,
     query: &str,
     page: u32,
-    search_settings: &server_models::Cookie,
+    search_settings: &server_models::Cookie<'_>,
 ) -> Result<(SearchResults, String, bool), Box<dyn std::error::Error>> {
     // eagerly parse cookie value to evaluate safe search level
     let safe_search_level = search_settings.safe_search_level;