aboutsummaryrefslogtreecommitdiff
path: root/src/web
diff options
context:
space:
mode:
authorQuentin <quentin@deuxfleurs.fr>2020-11-08 16:02:16 +0100
committerQuentin <quentin@deuxfleurs.fr>2020-11-08 16:02:16 +0100
commitc78df603d7355563c9f726f97bf318273fc5bb83 (patch)
tree6bb39cff50a656f98f1e21032acbeeacd5a92c43 /src/web
parent71721f5bcf943dcc3ffe0c3db02add4af335ab7a (diff)
downloadgarage-c78df603d7355563c9f726f97bf318273fc5bb83.tar.gz
garage-c78df603d7355563c9f726f97bf318273fc5bb83.zip
Add some documentation
Diffstat (limited to 'src/web')
-rw-r--r--src/web/web_server.rs10
1 files changed, 9 insertions, 1 deletions
diff --git a/src/web/web_server.rs b/src/web/web_server.rs
index a615ec8f..ce1f7ee1 100644
--- a/src/web/web_server.rs
+++ b/src/web/web_server.rs
@@ -58,7 +58,15 @@ async fn handler(
Ok(Response::new(Body::from("hello world\n")))
}
+/// Extract host from the authority section given by the HTTP host header
+///
+/// The HTTP host contains both a host and a port.
+/// Extracting the port is more complex than just finding the colon (:) symbol.
+/// An example of a case where it does not work: [::1]:3902
+/// Instead, we use the Uri module provided by Hyper that correctl parses this "authority" section
fn authority_to_host(authority: &str) -> Result<String, Error> {
+ // Hyper can not directly parse authority section so we build a fake URL
+ // that contains our authority section
let mut uri_str: String = "fake://".to_owned();
uri_str.push_str(authority);
@@ -66,7 +74,7 @@ fn authority_to_host(authority: &str) -> Result<String, Error> {
Ok(uri) => {
let host = uri
.host()
- .ok_or(Error::BadRequest(format!("Unable to extract host from authority as string")))?;
+ .ok_or(Error::BadRequest(format!("Unable to extract host from authority")))?;
Ok(String::from(host))
}
_ => Err(Error::BadRequest(format!("Unable to parse authority (host HTTP header)"))),