fix: Remove a few panics

Those panics should not crash the node. Folding them in the error case of their immediately enclosing `Result`.

crates/sui-core/src/authority_server.rs

@@ -57,7 +57,9 @@ impl AuthorityServerHandle {
     }
 
     pub async fn kill(self) -> Result<(), std::io::Error> {
-        self.tx_cancellation.send(()).unwrap();
+        self.tx_cancellation.send(()).map_err(|_e| {
+            std::io::Error::new(io::ErrorKind::Other, "could not send cancellation signal!")
+        })?;
         self.handle
             .await?
             .map_err(|e| std::io::Error::new(std::io::ErrorKind::Other, e))?;

crates/sui-types/src/crypto.rs

@@ -621,7 +621,8 @@ where
 {
     fn from_signable_bytes(bytes: &[u8]) -> Result<Self, Error> {
         // Remove name tag before deserialization using BCS
-        let name = serde_name::trace_name::<Self>().expect("Self must be a struct or an enum");
+        let name = serde_name::trace_name::<Self>()
+            .ok_or_else(|| anyhow::anyhow!("Self should be a struct or an enum"))?;
         let name_byte_len = format!("{}::", name).bytes().len();
         Ok(bcs::from_bytes(&bytes[name_byte_len..])?)
     }