fix(transport) Do not panic when building and Endpoint with an invali… (

#438)

BREAKING CHANGE: `TryFrom` API has been changed.

tonic/src/transport/channel/endpoint.rs

@@ -50,6 +50,10 @@ impl Endpoint {
 
     /// Convert an `Endpoint` from a static string.
     ///
+    /// # Panics
+    ///
+    /// This function panics if the argument is an invalid URI.
+    ///
     /// ```
     /// # use tonic::transport::Endpoint;
     /// Endpoint::from_static("https://example.com");
@@ -306,24 +310,13 @@ impl TryFrom<String> for Endpoint {
 }
 
 impl TryFrom<&'static str> for Endpoint {
-    type Error = Never;
+    type Error = InvalidUri;
 
     fn try_from(t: &'static str) -> Result<Self, Self::Error> {
-        Ok(Self::from_static(t))
-    }
-}
-
-#[derive(Debug)]
-pub enum Never {}
-
-impl std::fmt::Display for Never {
-    fn fmt(&self, _: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
-        match *self {}
+        Self::from_shared(t.as_bytes())
     }
 }
 
-impl std::error::Error for Never {}
-
 impl fmt::Debug for Endpoint {
     fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
         f.debug_struct("Endpoint").finish()