negotiated_alpn now returns string instead of Vec<u8>

Author: MarnixKuijs

src/lib.rs

@@ -655,8 +655,11 @@ impl<S: io::Read + io::Write> TlsStream<S> {
     }
 
     /// Returns the negotiated ALPN protocols
-    pub fn negotiated_alpn(&self) -> Result<Option<Vec<u8>>> {
-        Ok(self.0.negotiated_alpn()?)
+    pub fn negotiated_alpn(&self) -> Result<Option<String>> {
+        Ok(self
+            .0
+            .negotiated_alpn()?
+            .map(|bytes| String::from_utf8(bytes).unwrap())) //negotiated_alpn always returns valid utf-8
     }
 
     /// Shuts down the TLS session.

src/test.rs

@@ -424,7 +424,7 @@ mod tests {
         let s = p!(TcpStream::connect("google.com:443"));
         let socket = p!(builder.connect("google.com", s));
 
-        assert_eq!(p!(socket.negotiated_alpn()), Some(b"h2".to_vec()));
+        assert_eq!(p!(socket.negotiated_alpn()).as_deref(), Some("h2"));
     }
 
     #[test]
@@ -433,7 +433,7 @@ mod tests {
         let s = p!(TcpStream::connect("google.com:443"));
         let socket = p!(builder.connect("google.com", s));
 
-        assert_eq!(p!(socket.negotiated_alpn()), None);
+        assert_eq!(p!(socket.negotiated_alpn()).as_deref(), None);
     }
 
     #[test]