Fix measuring duration for MP3

nukeop · Nov 16, 2023 · 3a758fe · 3a758fe
1 parent 27f37ab
commit 3a758fe
Show file tree

Hide file tree

Showing 3 changed files with 59 additions and 12 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/packages/scanner/Cargo.toml b/packages/scanner/Cargo.toml
@@ -18,6 +18,7 @@ lewton = "0.10.2"
 md5 = "0.7.0"
 metaflac = "0.2.5"
 mockall = "0.11.4"
+mp3-duration = "0.1.10"
 mp4ameta = "0.11.0"
 
 [dependencies.image]

diff --git a/packages/scanner/src/metadata.rs b/packages/scanner/src/metadata.rs
@@ -62,7 +62,12 @@ impl MetadataExtractor for Mp3MetadataExtractor {
         metadata.artist = tag.artist().map(|s| s.to_string());
         metadata.title = tag.title().map(|s| s.to_string());
         metadata.album = tag.album().map(|s| s.to_string());
-        metadata.duration = tag.duration();
+        let duration = mp3_duration::from_path(&path).map(|duration| duration.as_secs() as u32);
+
+        match duration {
+            Ok(duration) => metadata.duration = Some(duration),
+            Err(_) => metadata.duration = None,
+        }
         metadata.position = tag.track();
         metadata.disc = tag.disc();
         metadata.year = tag.year().map(|s| s as u32);