Remove catch-all redirect for UI

Author: lmgivelaccenture

josh-proxy/src/bin/josh-proxy.rs

@@ -1259,19 +1259,17 @@ async fn call_service(
             }
             pu
         } else {
-            let redirect_path = if path == "/" {
-                "/~/ui/".to_string()
+            if path == "/" {
+                let redirect_path = "/~/ui/".to_string();
+                return Ok(Response::builder()
+                    .status(hyper::StatusCode::FOUND)
+                    .header("Location", redirect_path)
+                    .body(Full::new(Bytes::new()))?);
             } else {
-                format!(
-                    "/~/ui/browse?repo={}.git&path=&filter=%3A%2F&rev=HEAD",
-                    path
-                )
+                return Ok(Response::builder()
+                    .status(hyper::StatusCode::NOT_FOUND)
+                    .body(Full::new(Bytes::new()))?);
             };
-
-            return Ok(Response::builder()
-                .status(hyper::StatusCode::FOUND)
-                .header("Location", redirect_path)
-                .body(Full::new(Bytes::new()))?);
         }
     };
 

tests/proxy/clone_invalid_url.t

@@ -24,9 +24,7 @@
   $ cd ${TESTTMP}
 
   $ git clone -q http://localhost:8002/xxx full_repo
-  fatal: unable to update url base from redirection:
-    asked for: http://localhost:8002/xxx/info/refs?service=git-upload-pack
-     redirect: http://localhost:8002/~/ui/browse?repo=/xxx/info/refs.git&path=&filter=%3A%2F&rev=HEAD
+  fatal: repository 'http://localhost:8002/xxx/' not found
   [128]
 
   $ bash ${TESTDIR}/destroy_test_env.sh