]> Untitled Git - lemmy.git/commitdiff
Remove cache headers. Fixes #1222
authorDessalines <tyhou13@gmx.com>
Thu, 22 Oct 2020 16:53:53 +0000 (11:53 -0500)
committerDessalines <tyhou13@gmx.com>
Thu, 22 Oct 2020 16:53:53 +0000 (11:53 -0500)
lemmy_utils/src/lib.rs
src/main.rs

index 19e7bb8399d8d449af720702b84ae037af700f3b..eecb7b2d7fc3e4c51dd52223769931a7f94e486d 100644 (file)
@@ -78,6 +78,4 @@ lazy_static! {
     Settings::get().hostname
   ))
   .unwrap();
-  pub static ref CACHE_CONTROL_REGEX: Regex =
-    Regex::new("^((text|image)/.+|application/javascript)$").unwrap();
 }
index dea60c0a77aa5ae78bf8715547cb0219bf62620f..811c920aa1ec75a10d2426ca68f5cc154c8decf0 100644 (file)
@@ -2,38 +2,23 @@
 extern crate diesel_migrations;
 
 use actix::prelude::*;
-use actix_web::{
-  body::Body,
-  dev::{Service, ServiceRequest, ServiceResponse},
-  http::{
-    header::{CACHE_CONTROL, CONTENT_TYPE},
-    HeaderValue,
-  },
-  *,
-};
+use actix_web::*;
 use diesel::{
   r2d2::{ConnectionManager, Pool},
   PgConnection,
 };
-use lazy_static::lazy_static;
 use lemmy_api::match_websocket_operation;
 use lemmy_apub::activity_queue::create_activity_queue;
 use lemmy_db::get_database_url_from_env;
 use lemmy_rate_limit::{rate_limiter::RateLimiter, RateLimit};
 use lemmy_server::{code_migrations::run_advanced_migrations, routes::*};
 use lemmy_structs::blocking;
-use lemmy_utils::{settings::Settings, LemmyError, CACHE_CONTROL_REGEX};
+use lemmy_utils::{settings::Settings, LemmyError};
 use lemmy_websocket::{chat_server::ChatServer, LemmyContext};
 use reqwest::Client;
 use std::sync::Arc;
 use tokio::sync::Mutex;
 
-lazy_static! {
-  // static ref CACHE_CONTROL_VALUE: String = format!("public, max-age={}", 365 * 24 * 60 * 60);
-  // Test out 1 hour here, this is breaking some things
-  static ref CACHE_CONTROL_VALUE: String = format!("public, max-age={}", 60 * 60);
-}
-
 embed_migrations!();
 
 #[actix_web::main]
@@ -90,7 +75,6 @@ async fn main() -> Result<(), LemmyError> {
     );
     let rate_limiter = rate_limiter.clone();
     App::new()
-      .wrap_fn(add_cache_headers)
       .wrap(middleware::Logger::default())
       .data(context)
       // The routes
@@ -108,23 +92,3 @@ async fn main() -> Result<(), LemmyError> {
 
   Ok(())
 }
-
-fn add_cache_headers<S>(
-  req: ServiceRequest,
-  srv: &mut S,
-) -> impl Future<Output = Result<ServiceResponse, Error>>
-where
-  S: Service<Request = ServiceRequest, Response = ServiceResponse<Body>, Error = Error>,
-{
-  let fut = srv.call(req);
-  async move {
-    let mut res = fut.await?;
-    if let Some(content_type) = res.headers().get(CONTENT_TYPE) {
-      if CACHE_CONTROL_REGEX.is_match(content_type.to_str().unwrap()) {
-        let header_val = HeaderValue::from_static(&CACHE_CONTROL_VALUE);
-        res.headers_mut().insert(CACHE_CONTROL, header_val);
-      }
-    }
-    Ok(res)
-  }
-}