Library now panics! on invalid magnet url

Another helpful redditor warned me about the dangers of not validating
that the URL given is a magnet url. Unfortuanetly, I can't just change
everything to use Options and Errors, so I'm making the fix of having
the library just panic instead, since that would make it still
backwards compatible. This will be changed in version 2.0.0

README.md

@@ -1,5 +1,4 @@
 [![Build Status](https://www.travis-ci.com/billyb2/parse-magnet-rs.svg?branch=main)](https://www.travis-ci.com/billyb2/parse-magnet-rs)
-# The Rust Magnet URL Parser!
 # What is a magnet url
 A magnet is a URI scheme that identifies files by their hash,
 normally used in peer to peer file sharing networks (like
@@ -57,7 +56,6 @@ parameters!
 
  ```
  use magnet_url::Magnet;
- let magneturl =
  //Note, this magnet won't actually download, sorry :/
  Magnet {
      dn: "hello_world".to_string(),
@@ -67,13 +65,19 @@ parameters!
      tr:
          {
              let mut tr_vec = Vec::new();
-             tr_vec.push("https://example.com/".to_string())
+             tr_vec.push("https://example.com/".to_string());
              tr_vec
          },
      kt: "cool+stuff".to_string(),
      ws: String::new(),
      acceptable_source: String::new(),
      mt: String::new(),
-
+     xs: String::new(),
  };
  ```
+
+Invalid magnet url's will result in a panic! (This will be changed to an error in v2.0.0
+ ```#[should_panic]
+ use magnet_url::Magnet;
+ let _magnet_link = Magnet::new("https://example.com");
+ ```

src/lib.rs

@@ -4,6 +4,7 @@ use regex::Regex;
 extern crate lazy_static;
 
 ///The regexes used to identify specific parts of the magnet
+const MAGNET_URL_RE_STR: &str = r"^(stratum-|)magnet:\?";
 const DISPLAY_NAME_RE_STR: &str = r"dn=([A-Za-z0-9!@#$%^:*<>,?/()_+=.{}\{}\-]*)(&|$|\s)";
 const EXACT_TOPIC_RE_STR: &str = r"xt=urn:(sha1|btih|ed2k|aich|kzhash|md5|tree:tiger):([A-Fa-f0-9]+|[A-Za-z2-7]+)";
 const ADDRESS_TRACKER_RE_STR: &str = r"tr=([A-Za-z0-9!@#$%^:*<>,?/()_+=.{}\{}\-]*)(&|$|\s)";
@@ -71,7 +72,6 @@ const MANIFEST_TOPIC_RE_STR: &str = r"mt=((\w+)[A-Za-z0-9!@#$%^:*<>,?/()_+=.{}\\
 ///
 /// ```
 /// use magnet_url::Magnet;
-/// let magneturl =
 /// //Note, this magnet won't actually download, sorry :/
 /// Magnet {
 ///     dn: "hello_world".to_string(),
@@ -91,6 +91,12 @@ const MANIFEST_TOPIC_RE_STR: &str = r"mt=((\w+)[A-Za-z0-9!@#$%^:*<>,?/()_+=.{}\\
 ///     xs: String::new(),
 /// };
 /// ```
+///
+/// Invalid magnet url's will result in a panic! (This will be changed to an error in v2.0.0
+/// ```#[should_panic]
+/// use magnet_url::Magnet;
+/// let _magnet_link = Magnet::new("https://example.com");
+/// ```
 pub struct Magnet {
     ///Display Name of the torrent
     pub dn: String,
@@ -134,6 +140,7 @@ impl Magnet {
     */
     pub fn new (magnet_str: &str) -> Magnet {
         lazy_static! {
+            static ref MAGNET_URL_RE: Regex = Regex::new(MAGNET_URL_RE_STR).unwrap();
             static ref DISPLAY_NAME_RE: Regex = Regex::new(DISPLAY_NAME_RE_STR).unwrap();
             static ref EXACT_TOPIC_RE: Regex = Regex::new(EXACT_TOPIC_RE_STR).unwrap();
             static ref EXACT_LENGTH_RE: Regex = Regex::new(EXACT_LENGTH_RE_STR).unwrap();
@@ -145,6 +152,11 @@ impl Magnet {
             static ref MANIFEST_TOPIC_RE: Regex = Regex::new(MANIFEST_TOPIC_RE_STR).unwrap();
         }
 
+        // Panicking is a temporary fix, in version 2.0.0 it will instead return an Error
+        if MAGNET_URL_RE.is_match(magnet_str) == false {
+            panic!("Invalid magnet url")
+        }
+
         let validate_regex = |regex: &Regex, re_group_index| -> String {
             match regex.captures(magnet_str) {
                 Some(m) => m.get(re_group_index).map_or("", |m| m.as_str()).to_string(),
@@ -206,4 +218,11 @@ mod tests {
         assert_eq!(magnet_link.acceptable_source, String::new());
         assert_eq!(magnet_link.mt, String::new());
     }
+
+    #[test]
+    #[should_panic]
+    fn invalid_magnet_test() {
+        let _magnet_link = Magnet::new("https://example.com");
+
+    }
 }