mysten-network: default to http if not present

crates/mysten-network/src/multiaddr.rs

@@ -260,13 +260,10 @@ pub(crate) fn parse_tcp<'a, T: Iterator<Item = Protocol<'a>>>(protocols: &mut T)
 pub(crate) fn parse_http_https<'a, T: Iterator<Item = Protocol<'a>>>(
     protocols: &mut T,
 ) -> Result<&'static str> {
-    match protocols
-        .next()
-        .ok_or_else(|| eyre!("unexpected end of multiaddr"))?
-    {
-        Protocol::Http => Ok("http"),
-        Protocol::Https => Ok("https"),
-        _ => Err(eyre!("expected http/https protocol")),
+    match protocols.next() {
+        Some(Protocol::Http) => Ok("http"),
+        Some(Protocol::Https) => Ok("https"),
+        _ => Ok("http"),
     }
 }
 

crates/mysten-network/src/server.rs

@@ -467,13 +467,6 @@ mod test {
         test_multiaddr(address).await;
         std::fs::remove_file(path).unwrap();
     }
-
-    #[should_panic]
-    #[tokio::test]
-    async fn missing_http_protocol() {
-        let address: Multiaddr = "/dns/localhost/tcp/0".parse().unwrap();
-        test_multiaddr(address).await;
-    }
 }
 
 #[derive(Clone)]