Merge pull request #99 from 4t145/remove-default-log-level

Remove default log level in log config

tardis/src/basic/tracing.rs

@@ -127,7 +127,9 @@ where
     {
         self.with_configurable_layer(EnvFilter::from_default_env().boxed(), |config: &LogConfig| {
             let mut env_filter = EnvFilter::from_default_env();
-            env_filter = env_filter.add_directive(config.level.clone());
+            if let Some(level) = config.level.clone() {
+                env_filter = env_filter.add_directive(level);
+            }
             for directive in &config.directives {
                 env_filter = env_filter.add_directive(directive.clone());
             }

tardis/src/config/config_dto/log.rs

@@ -26,10 +26,10 @@ pub use tracing_appender::*;
 #[derive(Debug, Serialize, Deserialize, Clone, PartialEq, Eq, TypedBuilder)]
 #[serde(default)]
 pub struct LogConfig {
-    #[builder(default = "info".parse::<Directive>().expect(""), setter(into))]
+    #[builder(default, setter(into))]
     #[serde(deserialize_with = "deserialize_directive", serialize_with = "serialize_directive")]
     /// the defualt log level
-    pub level: Directive,
+    pub level: Option<Directive>,
     #[builder(default, setter(into))]
     #[serde(deserialize_with = "deserialize_directives", serialize_with = "serialize_directives")]
     /// tracing filtering directive, e.g. `tardis=debug,sqlx=off`
@@ -48,18 +48,18 @@ pub struct LogConfig {
     pub ext: HashMap<String, crate::serde_json::Value>,
 }
 
-fn serialize_directive<S>(value: &Directive, serializer: S) -> Result<S::Ok, S::Error>
+fn serialize_directive<S>(value: &Option<Directive>, serializer: S) -> Result<S::Ok, S::Error>
 where
     S: serde::Serializer,
 {
-    value.to_string().serialize(serializer)
+    value.as_ref().map(|d| d.to_string()).serialize(serializer)
 }
 
-fn deserialize_directive<'de, D>(deserializer: D) -> Result<Directive, D::Error>
+fn deserialize_directive<'de, D>(deserializer: D) -> Result<Option<Directive>, D::Error>
 where
     D: serde::Deserializer<'de>,
 {
-    String::deserialize(deserializer)?.parse::<Directive>().map_err(serde::de::Error::custom)
+    <Option<String>>::deserialize(deserializer)?.map(|s| s.parse::<Directive>().map_err(serde::de::Error::custom)).transpose()
 }
 fn serialize_directives<S>(value: &[Directive], serializer: S) -> Result<S::Ok, S::Error>
 where