actix/actix-extras
1
0
Fork 0
mirror of https://github.com/actix/actix-extras.git synced 2024-11-28 09:42:40 +01:00
Code Issues Releases Wiki Activity

Fix duplicate tail of StaticFiles with index_file

Browse Source 
Map from 0.6 to master
This commit is contained in:
Douman 2018-06-25 19:41:23 +03:00
parent 800c404c72
commit a9425a866b
1 changed files with 5 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
src/fs.rs
View File

@ -676,10 +676,6 @@ impl<S: 'static> Handler<S> for StaticFiles<S> {
// TODO: It'd be nice if there were a good usable URL manipulation // TODO: It'd be nice if there were a good usable URL manipulation
// library // library
let mut new_path: String = req.path().to_owned(); let mut new_path: String = req.path().to_owned();
for el in relpath.iter() {
new_path.push_str(&el.to_string_lossy());
new_path.push('/');
}
if !new_path.ends_with('/') { if !new_path.ends_with('/') {
new_path.push('/'); new_path.push('/');
} }
@ -1205,8 +1201,7 @@ mod tests {
#[test] #[test]
fn test_redirect_to_index() { fn test_redirect_to_index() {
let st = StaticFiles::new(".").index_file("index.html"); let st = StaticFiles::new(".").index_file("index.html");
let mut req = HttpRequest::default(); let req = TestRequest::default().uri("/tests").finish();
req.match_info_mut().add_static("tail", "tests");
let resp = st.handle(req).respond_to(&HttpRequest::default()).unwrap(); let resp = st.handle(req).respond_to(&HttpRequest::default()).unwrap();
let resp = resp.as_msg(); let resp = resp.as_msg();
@ -1216,8 +1211,7 @@ mod tests {
"/tests/index.html" "/tests/index.html"
); );
let mut req = HttpRequest::default(); let req = TestRequest::default().uri("/tests/").finish();
req.match_info_mut().add_static("tail", "tests/");
let resp = st.handle(req).respond_to(&HttpRequest::default()).unwrap(); let resp = st.handle(req).respond_to(&HttpRequest::default()).unwrap();
let resp = resp.as_msg(); let resp = resp.as_msg();
@ -1230,16 +1224,15 @@ mod tests {
#[test] #[test]
fn test_redirect_to_index_nested() { fn test_redirect_to_index_nested() {
let st = StaticFiles::new(".").index_file("mod.rs"); let st = StaticFiles::new(".").index_file("Cargo.toml");
let mut req = HttpRequest::default(); let req = TestRequest::default().uri("/tools/wsload").finish();
req.match_info_mut().add_static("tail", "src/client");
let resp = st.handle(req).respond_to(&HttpRequest::default()).unwrap(); let resp = st.handle(req).respond_to(&HttpRequest::default()).unwrap();
let resp = resp.as_msg(); let resp = resp.as_msg();
assert_eq!(resp.status(), StatusCode::FOUND); assert_eq!(resp.status(), StatusCode::FOUND);
assert_eq!( assert_eq!(
resp.headers().get(header::LOCATION).unwrap(), resp.headers().get(header::LOCATION).unwrap(),
"/src/client/mod.rs" "/tools/wsload/Cargo.toml"
); );
} }