Add setting for matrix sdk media caching

Author: VAWVAW

docs/iamb.5

@@ -133,6 +133,13 @@ The room to show by default instead of the
 .Sy :welcome
 window.
 
+.IT Sy cache_policy
+Configure how downloaded files and image previews are cached internally.
+Specifying this object will replace the upstream defaults and possibly lead to unconstrained cache growth.
+Possible keys and their effect are documented at:
+.br
+.Lk https://docs.rs/matrix-sdk/latest/matrix_sdk/media/struct.MediaRetentionPolicy.html#fields
+
 .It Sy image_preview
 Enable image previews and configure it.
 An empty object will enable the feature with default settings, omitting it will disable the feature.

src/config.rs

@@ -12,6 +12,7 @@ use std::process;
 
 use clap::Parser;
 use matrix_sdk::authentication::matrix::MatrixSession;
+use matrix_sdk::media::MediaRetentionPolicy;
 use matrix_sdk::ruma::{OwnedDeviceId, OwnedRoomAliasId, OwnedRoomId, OwnedUserId, UserId};
 use ratatui::style::{Color, Modifier as StyleModifier, Style};
 use ratatui::text::Span;
@@ -584,6 +585,7 @@ pub struct TunableValues {
     pub user_gutter_width: usize,
     pub external_edit_file_suffix: String,
     pub tabstop: usize,
+    pub cache_policy: MediaRetentionPolicy,
 }
 
 #[derive(Clone, Default, Deserialize)]
@@ -612,6 +614,7 @@ pub struct Tunables {
     pub user_gutter_width: Option<usize>,
     pub external_edit_file_suffix: Option<String>,
     pub tabstop: Option<usize>,
+    pub cache_policy: Option<MediaRetentionPolicy>,
 }
 
 impl Tunables {
@@ -646,6 +649,7 @@ impl Tunables {
                 .external_edit_file_suffix
                 .or(other.external_edit_file_suffix),
             tabstop: self.tabstop.or(other.tabstop),
+            cache_policy: self.cache_policy.or(other.cache_policy),
         }
     }
 
@@ -676,6 +680,7 @@ impl Tunables {
                 .external_edit_file_suffix
                 .unwrap_or_else(|| ".md".to_string()),
             tabstop: self.tabstop.unwrap_or(4),
+            cache_policy: self.cache_policy.unwrap_or_default(),
         }
     }
 }

src/tests.rs

@@ -200,6 +200,7 @@ pub fn mock_tunables() -> TunableValues {
         image_preview: None,
         user_gutter_width: 30,
         tabstop: 4,
+        cache_policy: Default::default(),
     }
 }
 

src/worker.rs

@@ -13,7 +13,6 @@ use std::time::{Duration, Instant};
 
 use futures::{stream::FuturesUnordered, StreamExt};
 use gethostname::gethostname;
-use matrix_sdk::media::MediaRetentionPolicy;
 use matrix_sdk::ruma::events::room::MediaSource;
 use ratatui_image::picker::Picker;
 use tokio::sync::mpsc::{unbounded_channel, UnboundedReceiver, UnboundedSender};
@@ -748,13 +747,7 @@ pub async fn create_client(settings: &ApplicationSettings) -> Client {
 
     client
         .media()
-        .set_media_retention_policy(
-            MediaRetentionPolicy::new()
-                // 4 GiB.
-                .with_max_cache_size(Some(4 * 1024 * 1024 * 1024))
-                // 200 MiB.
-                .with_max_file_size(Some(200 * 1024 * 1024)),
-        )
+        .set_media_retention_policy(settings.tunables.cache_policy)
         .await
         .expect("Failed to set cache policy");