Add workaround for incorrect hardsub labeling (#231)

crunchy-cli-core/Cargo.toml

@@ -13,7 +13,7 @@ anyhow = "1.0"
 async-trait = "0.1"
 clap = { version = "4.3", features = ["derive", "string"] }
 chrono = "0.4"
-crunchyroll-rs = { version = "0.5.0", features = ["dash-stream"] }
+crunchyroll-rs = { version = "0.5.1", features = ["dash-stream"] }
 ctrlc = "3.4"
 dialoguer = { version = "0.10", default-features = false }
 dirs = "5.0"

crunchy-cli-core/src/utils/video.rs

@@ -1,11 +1,32 @@
 use anyhow::Result;
 use crunchyroll_rs::media::{Resolution, Stream, VariantData};
+use crunchyroll_rs::Locale;
 
 pub async fn variant_data_from_stream(
     stream: &Stream,
     resolution: &Resolution,
 ) -> Result<Option<(VariantData, VariantData)>> {
-    let mut streaming_data = stream.dash_streaming_data(None).await?;
+    // sometimes Crunchyroll marks episodes without real subtitles that they have subtitles and
+    // reports that only hardsub episode are existing. the following lines are trying to prevent
+    // potential errors which might get caused by this incorrect reporting
+    // (https://github.com/crunchy-labs/crunchy-cli/issues/231)
+    let mut hardsub_locales = stream.streaming_hardsub_locales();
+    let hardsub_locale = if !hardsub_locales.contains(&Locale::Custom("".to_string()))
+        && !hardsub_locales.contains(&Locale::Custom(":".to_string()))
+    {
+        // if only one hardsub locale exists, assume that this stream doesn't really contains hardsubs
+        if hardsub_locales.len() == 1 {
+            Some(hardsub_locales.remove(0))
+        } else {
+            // fallback to `None`. this should trigger an error message in `stream.dash_streaming_data`
+            // that the requested stream is not available
+            None
+        }
+    } else {
+        None
+    };
+
+    let mut streaming_data = stream.dash_streaming_data(hardsub_locale).await?;
     streaming_data
         .0
         .sort_by(|a, b| a.bandwidth.cmp(&b.bandwidth).reverse());