From 6dfe4d1c9b08b108ad71945a60ba24256b61ba26 Mon Sep 17 00:00:00 2001 From: Billy Bradley Date: Sat, 1 May 2021 21:36:55 +0000 Subject: Add --spa-index option --- src/main.rs | 47 +++++++++++++++++++++++++++++++++++++---------- 1 file changed, 37 insertions(+), 10 deletions(-) (limited to 'src/main.rs') diff --git a/src/main.rs b/src/main.rs index 133f320..f3dc18a 100644 --- a/src/main.rs +++ b/src/main.rs @@ -1,6 +1,7 @@ use std::io; use std::io::Write; use std::net::{IpAddr, SocketAddr, TcpListener}; +use std::path::PathBuf; use std::thread; use std::time::Duration; @@ -105,15 +106,9 @@ async fn run(miniserve_config: MiniserveConfig) -> Result<(), ContextualError> { ContextualError::IoError("Failed to resolve path to be served".to_string(), e) })?; - if let Some(index_path) = &miniserve_config.index { - let has_index: std::path::PathBuf = [&canon_path, index_path].iter().collect(); - if !has_index.exists() { - error!( - "The file '{}' provided for option --index could not be found.", - index_path.to_string_lossy() - ); - } - } + check_file_exists(&canon_path, &miniserve_config.index, "index"); + check_file_exists(&canon_path, &miniserve_config.spa_index, "spa-index"); + let path_string = canon_path.to_string_lossy(); println!( @@ -279,6 +274,19 @@ async fn run(miniserve_config: MiniserveConfig) -> Result<(), ContextualError> { .map_err(|e| ContextualError::IoError("".to_owned(), e)) } +fn check_file_exists(canon_path: &PathBuf, file_option: &Option, option_name: &str) { + if let Some(file_path) = file_option { + let has_file: std::path::PathBuf = [&canon_path, file_path].iter().collect(); + if !has_file.exists() { + error!( + "The file '{}' provided for option --{} could not be found.", + file_path.to_string_lossy(), + option_name, + ); + } + } +} + /// Allows us to set low-level socket options /// /// This mainly used to set `set_only_v6` socket option @@ -311,6 +319,26 @@ fn configure_app(app: &mut web::ServiceConfig, conf: &MiniserveConfig) { Some(index_file) => files.index_file(index_file.to_string_lossy()), None => files, }; + let files = match &conf.spa_index { + Some(spa_index_file) => { + let spa_index_full = &conf.path.join(spa_index_file); + let spa_index_string: String = spa_index_full.to_string_lossy().into(); + + files.default_handler(move |req: actix_web::dev::ServiceRequest| { + let (request, _payload) = req.into_parts(); + let spa_index_string = spa_index_string.clone(); + + async move { + let response = actix_files::NamedFile::open( + &spa_index_string + )? + .into_response(&request); + Ok(actix_web::dev::ServiceResponse::new(request, response)) + } + }) + }, + None => files.default_handler(web::to(error_404)), + }; let files = match conf.show_hidden { true => files.use_hidden_files(), false => files, @@ -320,7 +348,6 @@ fn configure_app(app: &mut web::ServiceConfig, conf: &MiniserveConfig) { .files_listing_renderer(listing::directory_listing) .prefer_utf8(true) .redirect_to_slash_directory() - .default_handler(web::to(error_404)) }; if !conf.path.is_file() { -- cgit v1.2.3 From 9772505ccb2499a8f533d649b8334fd9bf765631 Mon Sep 17 00:00:00 2001 From: Billy Bradley Date: Sun, 10 Oct 2021 14:13:43 +0100 Subject: Use NamedFile as default handler to simplify --spa-index option implementation --- src/main.rs | 22 +++++----------------- 1 file changed, 5 insertions(+), 17 deletions(-) (limited to 'src/main.rs') diff --git a/src/main.rs b/src/main.rs index f3dc18a..7420888 100644 --- a/src/main.rs +++ b/src/main.rs @@ -5,6 +5,7 @@ use std::path::PathBuf; use std::thread; use std::time::Duration; +use actix_files::NamedFile; use actix_web::web; use actix_web::{http::header::ContentType, Responder}; use actix_web::{middleware, App, HttpRequest, HttpResponse}; @@ -320,23 +321,10 @@ fn configure_app(app: &mut web::ServiceConfig, conf: &MiniserveConfig) { None => files, }; let files = match &conf.spa_index { - Some(spa_index_file) => { - let spa_index_full = &conf.path.join(spa_index_file); - let spa_index_string: String = spa_index_full.to_string_lossy().into(); - - files.default_handler(move |req: actix_web::dev::ServiceRequest| { - let (request, _payload) = req.into_parts(); - let spa_index_string = spa_index_string.clone(); - - async move { - let response = actix_files::NamedFile::open( - &spa_index_string - )? - .into_response(&request); - Ok(actix_web::dev::ServiceResponse::new(request, response)) - } - }) - }, + Some(spa_index_file) => files.default_handler( + NamedFile::open(&conf.path.join(spa_index_file)) + .expect("Cant open SPA index file.") + ), None => files.default_handler(web::to(error_404)), }; let files = match conf.show_hidden { -- cgit v1.2.3 From 6090cd5e297c5b54ee385b0039794607e809e029 Mon Sep 17 00:00:00 2001 From: Billy Bradley Date: Thu, 14 Oct 2021 12:46:51 +0100 Subject: Fix formatting --- src/main.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/main.rs') diff --git a/src/main.rs b/src/main.rs index 7420888..32fcdec 100644 --- a/src/main.rs +++ b/src/main.rs @@ -323,7 +323,7 @@ fn configure_app(app: &mut web::ServiceConfig, conf: &MiniserveConfig) { let files = match &conf.spa_index { Some(spa_index_file) => files.default_handler( NamedFile::open(&conf.path.join(spa_index_file)) - .expect("Cant open SPA index file.") + .expect("Cant open SPA index file."), ), None => files.default_handler(web::to(error_404)), }; -- cgit v1.2.3 From 26e89415f34cc17d8621b21e5802738eefcb3535 Mon Sep 17 00:00:00 2001 From: Billy Bradley Date: Mon, 18 Oct 2021 15:00:15 +0100 Subject: Use Path instead of PathBuf for parameter --- src/main.rs | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) (limited to 'src/main.rs') diff --git a/src/main.rs b/src/main.rs index 32fcdec..eb159a2 100644 --- a/src/main.rs +++ b/src/main.rs @@ -1,7 +1,7 @@ use std::io; use std::io::Write; use std::net::{IpAddr, SocketAddr, TcpListener}; -use std::path::PathBuf; +use std::path::{Path, PathBuf}; use std::thread; use std::time::Duration; @@ -275,9 +275,10 @@ async fn run(miniserve_config: MiniserveConfig) -> Result<(), ContextualError> { .map_err(|e| ContextualError::IoError("".to_owned(), e)) } -fn check_file_exists(canon_path: &PathBuf, file_option: &Option, option_name: &str) { +fn check_file_exists(canon_path: &Path, file_option: &Option, option_name: &str) { if let Some(file_path) = file_option { - let has_file: std::path::PathBuf = [&canon_path, file_path].iter().collect(); + let file_path: &Path = file_path.as_ref(); + let has_file: std::path::PathBuf = [canon_path, file_path].iter().collect(); if !has_file.exists() { error!( "The file '{}' provided for option --{} could not be found.", -- cgit v1.2.3