fix: MagnetLink equality is not sensitive to query param order

Author: angrynode

CHANGELOG.md

@@ -22,6 +22,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - `MagnetFile` sorts and deduplicates the trackers provided in the magnet URL;
   `MagnetFile::trackers` returns them in an order, which is not specified
 
+### Fixed
+
+- `MagnetLink` equality is no longer affected by the exact representation of the
+  magnet URL; if the name, hash, and (sorted/deduped) trackers are equal, then
+  two magnets are considered equal
+
 ## Version 0.4.0 (2025-11-10)
 
 This release is focused on stricter parsing of torrents and magnets, and

src/magnet.rs

@@ -338,7 +338,14 @@ impl std::fmt::Display for MagnetLink {
 
 impl PartialEq for MagnetLink {
     fn eq(&self, other: &Self) -> bool {
-        self.query == other.query
+        // Here we may have different ordering of the URL params so we check the
+        // values. This is more expensive but more correct.
+        //
+        // In the future, we may optimize this by reproducing a normalized query.
+        self.name == other.name
+            && self.hash == other.hash
+            // Trackers have been sorted in the constructor
+            && self.trackers == other.trackers
     }
 }