Revert "fix: Return error on non https uri instead of panic (hyperium…

…#838)"

This reverts commit ef6e245.

tonic/src/transport/service/connector.rs

@@ -5,7 +5,6 @@ use super::tls::TlsConnector;
 use http::Uri;
 #[cfg(feature = "tls-roots-common")]
 use std::convert::TryInto;
-use std::fmt;
 use std::task::{Context, Poll};
 use tower::make::MakeConnection;
 use tower_service::Service;
@@ -79,43 +78,20 @@ where
         #[cfg(feature = "tls-roots-common")]
         let tls = self.tls_or_default(uri.scheme_str(), uri.host());
 
-        let is_https = uri.scheme_str() == Some("https");
         let connect = self.inner.make_connection(uri);
 
         Box::pin(async move {
-            #[cfg(not(feature = "tls"))]
-            {
-                if is_https {
-                    return Err(HttpsUriWithoutTlsSupport(()).into());
-                }
-            }
-
             let io = connect.await?;
 
             #[cfg(feature = "tls")]
             {
                 if let Some(tls) = tls {
                     let conn = tls.connect(io).await?;
                     return Ok(BoxedIo::new(conn));
-                } else if is_https {
-                    return Err(HttpsUriWithoutTlsSupport(()).into());
                 }
             }
 
             Ok(BoxedIo::new(io))
         })
     }
 }
-
-/// Error returned when trying to connect to an HTTPS endpoint without TLS enabled.
-#[derive(Debug)]
-pub(crate) struct HttpsUriWithoutTlsSupport(());
-
-impl fmt::Display for HttpsUriWithoutTlsSupport {
-    fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
-        write!(f, "Connecting to HTTPS without TLS enabled")
-    }
-}
-
-// std::error::Error only requires a type to impl Debug and Display
-impl std::error::Error for HttpsUriWithoutTlsSupport {}