Net: work around handling of empty PathAndQuery in http.

An absolute http::uri::Uri with an empty but set PathAndQuery ("") will
pretend that the path is "/" for comparison, display or HTTP request
construction purposes. However, a relative URI constructed from such
PathAndQuery will be empty.

A directory URI without a path (i.e. "https://example.com") thus
resulted in an empty path in the request line and 400 Bad Request from
the server.

Fixes #97.

Author: bavshin-f5

Cargo.lock

@@ -13,7 +13,7 @@ crate-type = ["cdylib"]
 base64 = "0.22.1"
 bytes = "1.10.1"
 constcat = "0.6.1"
-http = "1.3.1"
+http = "1.4.0"
 http-body = "1.0.1"
 http-body-util = "0.1.3"
 http-serde = "2.1.1"

Cargo.toml

@@ -107,18 +107,22 @@ impl HttpClient for NgxHttpClient<'_> {
         <B as Body>::Data: Send,
         <B as Body>::Error: StdError + Send + Sync,
     {
-        let uri = req.uri().clone();
+        const DEFAULT_PATH: http::uri::PathAndQuery = http::uri::PathAndQuery::from_static("/");
+
+        let path_and_query = req
+            .uri()
+            .path_and_query()
+            // filter empty ("") values that are represented as "/"
+            .filter(|x| x.as_str() != "/")
+            .cloned()
+            .unwrap_or(DEFAULT_PATH);
+
+        let uri = core::mem::replace(req.uri_mut(), path_and_query.into());
 
         let authority = uri
             .authority()
             .ok_or(HttpClientError::Uri("missing authority"))?;
 
-        let path_and_query = uri
-            .path_and_query()
-            .ok_or(HttpClientError::Uri("missing path"))?;
-
-        *req.uri_mut() = path_and_query.clone().into();
-
         {
             let headers = req.headers_mut();
             headers.insert(