changed default log file to ~/.odilia.log (#151)

* changed default log file to ~/.odilia.log * updated to use env instead of dirs home_dir, need to create odilia directory in .local/share * changed log file to XDG_DATA * removed TODOs * changed log path --------- Co-authored-by: Ellis Haker <[email protected]>
odilia-app · May 29, 2024 · f944d9d · f944d9d
1 parent 23797bd
commit f944d9d
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 6 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/common/Cargo.toml b/common/Cargo.toml
@@ -21,3 +21,4 @@ thiserror = "1.0.37"
 zbus.workspace = true
 serde_plain.workspace = true
 figment = "0.10.15"
+xdg = "2.4.1"
diff --git a/common/src/settings/log.rs b/common/src/settings/log.rs
@@ -1,6 +1,6 @@
+use serde::{Deserialize, Serialize};
 use std::path::PathBuf;
 
-use serde::{Deserialize, Serialize};
 ///structure used for all the configurable options related to logging
 #[derive(Debug, Serialize, Deserialize)]
 #[allow(clippy::module_name_repetitions)]
@@ -16,10 +16,14 @@ pub struct LogSettings {
 }
 impl Default for LogSettings {
 	fn default() -> Self {
-		Self {
-			level: "info".to_owned(),
-			logger: LoggingKind::File("/var/log/odilia.log".into()),
-		}
+		let xdg_dirs = xdg::BaseDirectories::with_prefix("odilia").expect(
+			"unable to find the odilia config directory according to the xdg dirs specification",
+		);
+		let log_path = xdg_dirs
+			.place_state_file("odilia.log")
+			.expect("unable to place log file");
+
+		Self { level: "info".to_owned(), logger: LoggingKind::File(log_path) }
 	}
 }