Change domain name to option

Author: dlon

installer-downloader/src/ui_downloader.rs

@@ -130,15 +130,15 @@ impl<Delegate: AppDelegate, Downloader: AppDownloader + Send + 'static> AppDownl
 
 /// Implementation of [fetch::ProgressUpdater] that updates some [AppDelegate].
 pub struct UiProgressUpdater<Delegate: AppDelegate> {
-    domain: String,
+    domain: Option<String>,
     prev_progress: Option<u32>,
     queue: Delegate::Queue,
 }
 
 impl<Delegate: AppDelegate> UiProgressUpdater<Delegate> {
     pub fn new(queue: Delegate::Queue) -> Self {
         Self {
-            domain: "unknown source".to_owned(),
+            domain: None,
             prev_progress: None,
             queue,
         }
@@ -161,9 +161,13 @@ impl<Delegate: AppDelegate> UiProgressUpdater<Delegate> {
         format!(
             "{} {}... ({complete_percentage}%)",
             resource::DOWNLOADING_DESC_PREFIX,
-            self.domain
+            self.domain()
         )
     }
+
+    fn domain(&self) -> &str {
+        self.domain.as_deref().unwrap_or("unknown source")
+    }
 }
 
 impl<Delegate: AppDelegate + 'static> fetch::ProgressUpdater for UiProgressUpdater<Delegate> {
@@ -201,6 +205,6 @@ impl<Delegate: AppDelegate + 'static> fetch::ProgressUpdater for UiProgressUpdat
         // Parse out domain name
         let url = url.strip_prefix("https://").unwrap_or(url);
         let (domain, _) = url.split_once('/').unwrap_or((url, ""));
-        self.domain = domain.to_owned();
+        self.domain = Some(domain.to_owned());
     }
 }