diff options
| author | Sven-Hendrik Haase <svenstaro@gmail.com> | 2024-01-03 01:53:59 +0000 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-01-03 01:53:59 +0000 |
| commit | ac1aa757a29e0c885576625a47aa5637f0b53c29 (patch) | |
| tree | 8c7ed410c9365953bd9fddd7f65e2685d1090fa0 | |
| parent | Merge pull request #1290 from 5long/remove-content-encoding (diff) | |
| parent | Prevent illegal request path from crashing program (diff) | |
| download | miniserve-ac1aa757a29e0c885576625a47aa5637f0b53c29.tar.gz miniserve-ac1aa757a29e0c885576625a47aa5637f0b53c29.zip | |
Merge pull request #1285 from cyqsimon/listing-crash-fix
Prevent illegal request path from crashing program
| -rw-r--r-- | src/listing.rs | 17 |
1 files changed, 11 insertions, 6 deletions
diff --git a/src/listing.rs b/src/listing.rs index eb9b2be..6e75b0e 100644 --- a/src/listing.rs +++ b/src/listing.rs @@ -162,12 +162,17 @@ pub fn directory_listing( let base = Path::new(serve_path); let random_route_abs = format!("/{}", conf.route_prefix); - let abs_uri = http::Uri::builder() - .scheme(req.connection_info().scheme()) - .authority(req.connection_info().host()) - .path_and_query(req.uri().to_string()) - .build() - .unwrap(); + let abs_uri = { + let res = http::Uri::builder() + .scheme(req.connection_info().scheme()) + .authority(req.connection_info().host()) + .path_and_query(req.uri().to_string()) + .build(); + match res { + Ok(uri) => uri, + Err(err) => return Ok(ServiceResponse::from_err(err, req.clone())), + } + }; let is_root = base.parent().is_none() || Path::new(&req.path()) == Path::new(&random_route_abs); let encoded_dir = match base.strip_prefix(random_route_abs) { |