Merge pull request #11 from gibbz00/ws_error_other

Report error rather than panicking with `unreachable!`
najamelan · Feb 4, 2025 · c70551f · c70551f
2 parents 4035d46 + 278e763
commit c70551f
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/src/error.rs b/src/error.rs
@@ -103,6 +103,11 @@ pub enum WsErr
 	#[ error( "Received a message that is neither ArrayBuffer, String or Blob." ) ]
 	//
 	UnknownDataType,
+
+    /// Fallback whenever explicit variant hadling hansn't yet been implemented.
+    #[error("`{0}`")]
+    //
+    Other(String),
 }
 
 
diff --git a/src/ws_meta.rs b/src/ws_meta.rs
@@ -83,7 +83,7 @@ impl WsMeta
 						return Err( WsErr::InvalidUrl{ supplied: url.as_ref().to_string() } ),
 
 
-					_ => unreachable!(),
+					_ => return Err(WsErr::Other(de.message())),
 				};
 			}
 		};
-Original file line number
+Diff line change
@@ Expand Up / @@ -83,7 +83,7 @@ impl WsMeta @@
     						return Err( WsErr::InvalidUrl{ supplied: url.as_ref().to_string() } ),
-    					_ => unreachable!(),
+    					_ => return Err(WsErr::Other(de.message())),
     				};
     			}
     		};
@@ Expand Down @@