Merge branch 'main' into add-kv-support

Author: estk

.github/workflows/coverage.yml

@@ -30,6 +30,7 @@ jobs:
         USER: "test-user"
 
     - name: Upload to codecov.io
-      uses: codecov/codecov-action@v3
+      uses: codecov/codecov-action@v5
       with:
-        fail_ci_if_error: true
+        token: ${{ secrets.CODECOV_TOKEN }}
+        verbose: true # optional (default = false)

.github/workflows/lint.yml

@@ -4,12 +4,12 @@ name: Lint Jobs
 on:
   push:
     branches:
-    - main
-    - devel
+      - main
+      - devel
   pull_request:
     branches:
-    - main
-    - devel
+      - main
+      - devel
 
 jobs:
   rust-lint:
@@ -27,30 +27,30 @@ jobs:
         run: cargo check
 
       - name: Run rustfmt
-        run: cargo fmt -- --check
+        run: cargo +nightly fmt -- --check
 
       - name: Run Clippy
-        run: cargo clippy --all-features
+        run: rustup component add --toolchain 1.75-x86_64-unknown-linux-gnu clippy && cargo +1.75 clippy --all-features
         env:
           RUSTFLAGS: -D warnings
 
   changelog-lint:
     name: Changelog Lint
     runs-on: ubuntu-latest
     steps:
-    - name: Markdown Linting Action
-      uses: avto-dev/markdown-lint@v1
-      with:
-        rules: /lint/rules/changelog.js
-        config: --config /lint/config/changelog.yml
-        args: 'CHANGELOG.md'
+      - name: Markdown Linting Action
+        uses: avto-dev/markdown-lint@v1
+        with:
+          rules: /lint/rules/changelog.js
+          config: --config /lint/config/changelog.yml
+          args: "CHANGELOG.md"
 
   docs-lint:
     name: Docs Lint
     runs-on: ubuntu-latest
     steps:
-    - name: Markdown Linting Action
-      uses: avto-dev/markdown-lint@v1
-      with:
-        args: '**.md'
-        ignore: 'CHANGELOG.md'
+      - name: Markdown Linting Action
+        uses: avto-dev/markdown-lint@v1
+        with:
+          args: "**.md"
+          ignore: "CHANGELOG.md"

.github/workflows/main.yml

@@ -17,7 +17,7 @@ jobs:
     runs-on: ${{ matrix.os }}
     strategy:
       matrix:
-        rust_versions: ["stable", "1.69"]
+        rust_versions: ["stable", "1.75"]
         os: [ubuntu-latest, windows-latest]
     steps:
     - name: Checkout the source code
@@ -36,7 +36,7 @@ jobs:
         # This test needs another run under the release mode
         cargo test --release --features simple_writer encode::pattern::tests::debug_release
       env:
-        RUSTFLAGS: -D warnings
+        RUSTFLAGS: -D warnings --cfg mock_time
 
   features:
     name: Test Individual Features
@@ -55,15 +55,15 @@ jobs:
       run: |
         for feature in $(cargo read-manifest | jq -r '.features|keys|join("\n")'); do
             echo building with feature "$feature"
-            RUSTFLAGS='-D warnings' cargo test --no-default-features --features "$feature"
+            RUSTFLAGS='-D warnings --cfg mock_time' cargo test --no-default-features --features "$feature"
         done
 
   bench:
     name: Benchmark the background_rotation feature
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        rust_versions: ["stable", "1.69"]
+        rust_versions: ["stable", "1.75"]
     steps:
     - name: Checkout the source code
       uses: actions/checkout@v4

CHANGELOG.md

@@ -1,5 +1,15 @@
 # Change Log
 
+## [Unreleased]
+
+### Changed
+
+* MSRV to 1.75
+* Update thread-id
+* Update thiserror
+* Update mock_instant
+* Update rand
+
 ## [1.3.0]
 
 ### New

Cargo.toml

@@ -11,11 +11,11 @@ repository = "https://github.com/estk/log4rs"
 readme = "README.md"
 keywords = ["log", "logger", "logging", "log4"]
 edition = "2018"
-rust-version = "1.69"
+rust-version = "1.75"
 
 [features]
 default = ["all_components", "config_parsing", "yaml_format"]
-
+onstartup_trigger = []
 config_parsing = [
     "humantime",
     "serde",
@@ -26,7 +26,6 @@ config_parsing = [
 yaml_format = ["serde_yaml"]
 json_format = ["serde_json"]
 toml_format = ["toml"]
-
 console_appender = ["console_writer", "simple_writer", "pattern_encoder"]
 file_appender = ["parking_lot", "simple_writer", "pattern_encoder"]
 rolling_file_appender = ["parking_lot", "simple_writer", "pattern_encoder"]
@@ -35,7 +34,6 @@ delete_roller = []
 fixed_window_roller = []
 size_trigger = []
 time_trigger = ["rand"]
-onstartup_trigger = []
 json_encoder = [
     "serde",
     "serde_json",
@@ -85,17 +83,18 @@ log = { version = "0.4.21", features = ["std"] }
 log-mdc = { version = "0.1", optional = true }
 serde = { version = "1.0.196", optional = true, features = ["derive"] }
 serde-value = { version = "0.7", optional = true }
-thread-id = { version = "4", optional = true }
+thread-id = { version = "5", optional = true }
 typemap-ors = { version = "1.0.0", optional = true }
 serde_json = { version = "1.0", optional = true }
 serde_yaml = { version = "0.9", optional = true }
 toml = { version = "<0.8.10", optional = true }
 parking_lot = { version = "0.12.0", optional = true }
-rand = { version = "0.8", optional = true }
-thiserror = "1.0.15"
+rand = { version = "0.9", optional = true }
+thiserror = "2.0.12"
 anyhow = "1.0.28"
 derivative = "2.2"
 once_cell = "1.17.1"
+mock_instant = "0.5"
 
 [target.'cfg(windows)'.dependencies]
 winapi = { version = "0.3", optional = true, features = [
@@ -114,7 +113,6 @@ lazy_static = "1.4"
 streaming-stats = "0.2.3"
 humantime = "2.1"
 tempfile = "3.8"
-mock_instant = "0.3"
 serde_test = "1.0.176"
 
 [[example]]
@@ -140,3 +138,6 @@ required-features = ["file_appender", "rolling_file_appender", "size_trigger"]
 [[example]]
 name = "multi_logger_config"
 required-features = ["yaml_format", "config_parsing"]
+
+[lints.rust]
+unexpected_cfgs = { level = "warn", check-cfg = ['cfg(mock_time)'] }

README.md

@@ -4,7 +4,7 @@
 [![crates.io](https://img.shields.io/crates/v/log4rs.svg)](https://crates.io/crates/log4rs)
 [![License: MIT OR Apache-2.0](https://img.shields.io/crates/l/clippy.svg)](#license)
 ![CI](https://github.com/estk/log4rs/workflows/CI/badge.svg)
-[![Minimum rustc version](https://img.shields.io/badge/rustc-1.69+-green.svg)](https://github.com/estk/log4rs#rust-version-requirements)
+[![Minimum rustc version](https://img.shields.io/badge/rustc-1.75+-green.svg)](https://github.com/estk/log4rs#rust-version-requirements)
 
 log4rs is a highly configurable logging framework modeled after Java's Logback
 and log4j libraries.
@@ -54,7 +54,7 @@ fn main() {
 
 ## Rust Version Requirements
 
-1.69
+1.75
 
 ## Building for Dev
 

src/append/rolling_file/policy/compound/trigger/time.rs

@@ -2,11 +2,8 @@
 //!
 //! Requires the `time_trigger` feature.
 
-#[cfg(test)]
-use chrono::NaiveDateTime;
 use chrono::{DateTime, Datelike, Duration, Local, TimeZone, Timelike};
-#[cfg(test)]
-use mock_instant::{SystemTime, UNIX_EPOCH};
+
 use rand::Rng;
 #[cfg(feature = "config_parsing")]
 use serde::de;
@@ -176,22 +173,23 @@ impl TimeTrigger {
     /// Returns a new trigger which rolls the log once it has passed the
     /// specified time.
     pub fn new(config: TimeTriggerConfig) -> TimeTrigger {
-        #[cfg(test)]
+        #[cfg(mock_time)]
         let current = {
+            use mock_instant::thread_local::{SystemTime, UNIX_EPOCH};
+
             let now: std::time::Duration = SystemTime::now()
                 .duration_since(UNIX_EPOCH)
                 .expect("system time before Unix epoch");
-            NaiveDateTime::from_timestamp_opt(now.as_secs() as i64, now.subsec_nanos())
-                .unwrap()
-                .and_local_timezone(Local)
+            DateTime::from_timestamp(now.as_secs() as i64, now.subsec_nanos())
                 .unwrap()
+                .with_timezone(&Local)
         };
 
-        #[cfg(not(test))]
+        #[cfg(not(mock_time))]
         let current = Local::now();
         let next_time = TimeTrigger::get_next_time(current, config.interval, config.modulate);
         let next_roll_time = if config.max_random_delay > 0 {
-            let random_delay = rand::thread_rng().gen_range(0..config.max_random_delay);
+            let random_delay = rand::rng().random_range(0..config.max_random_delay);
             next_time + Duration::seconds(random_delay as i64)
         } else {
             next_time
@@ -278,18 +276,19 @@ impl TimeTrigger {
 
 impl Trigger for TimeTrigger {
     fn trigger(&self, _file: &LogFile) -> anyhow::Result<bool> {
-        #[cfg(test)]
+        #[cfg(mock_time)]
         let current = {
+            use mock_instant::thread_local::{SystemTime, UNIX_EPOCH};
+
             let now = SystemTime::now()
                 .duration_since(UNIX_EPOCH)
                 .expect("system time before Unix epoch");
-            NaiveDateTime::from_timestamp_opt(now.as_secs() as i64, now.subsec_nanos())
-                .unwrap()
-                .and_local_timezone(Local)
+            DateTime::from_timestamp(now.as_secs() as i64, now.subsec_nanos())
                 .unwrap()
+                .with_timezone(&Local)
         };
 
-        #[cfg(not(test))]
+        #[cfg(not(mock_time))]
         let current: DateTime<Local> = Local::now();
         let mut next_roll_time = self.next_roll_time.write().unwrap();
         let is_trigger = current >= *next_roll_time;
@@ -340,7 +339,7 @@ impl Deserialize for TimeTriggerDeserializer {
 #[cfg(test)]
 mod test {
     use super::*;
-    use mock_instant::MockClock;
+    use mock_instant::thread_local::MockClock;
     use std::time::Duration;
 
     fn trigger_with_time_and_modulate(
@@ -373,6 +372,7 @@ mod test {
     }
 
     #[test]
+    #[ignore]
     fn trigger() {
         let second_in_milli = 1000;
         let minute_in_milli = second_in_milli * 60;
@@ -392,7 +392,8 @@ mod test {
             (TimeTriggerInterval::Year(1), year_in_milli),
         ];
         let modulate = false;
-        for (time_trigger_interval, time_in_milli) in test_list.iter() {
+        for (time_trigger_interval, time_in_milli) in &test_list {
+            dbg!(time_in_milli);
             MockClock::set_system_time(Duration::from_millis(4 * day_in_milli)); // 1970/1/5 00:00:00 Monday
             assert_eq!(
                 trigger_with_time_and_modulate(*time_trigger_interval, modulate, *time_in_milli),
@@ -418,7 +419,8 @@ mod test {
             (TimeTriggerInterval::Year(3), 3 * year_in_milli),
         ];
         let modulate = true;
-        for (time_trigger_interval, time_in_milli) in test_list.iter() {
+        for (time_trigger_interval, time_in_milli) in &test_list {
+            dbg!(time_in_milli);
             MockClock::set_system_time(Duration::from_millis(
                 59 * day_in_milli + 2 * hour_in_milli + 2 * minute_in_milli + 2 * second_in_milli,
             )); // 1970/3/1 02:02:02 Sunday

src/config/mod.rs

@@ -27,9 +27,7 @@ pub use self::raw::{Deserializable, Deserialize, Deserializers, RawConfig};
 pub fn init_config(config: runtime::Config) -> Result<crate::Handle, SetLoggerError> {
     let logger = crate::Logger::new(config);
     log::set_max_level(logger.max_log_level());
-    let handle = Handle {
-        shared: logger.0.clone(),
-    };
+    let handle = logger.handle();
     log::set_boxed_logger(Box::new(logger)).map(|()| handle)
 }
 

src/lib.rs

@@ -177,7 +177,7 @@
 //! For more examples see the [examples](https://github.com/estk/log4rs/tree/main/examples).
 //!
 
-#![allow(where_clauses_object_safety, clippy::manual_non_exhaustive)]
+#![allow(clippy::manual_non_exhaustive)]
 #![warn(missing_docs)]
 
 use std::{
@@ -414,6 +414,12 @@ impl Logger {
     pub fn max_log_level(&self) -> LevelFilter {
         self.0.load().root.max_log_level()
     }
+    /// Get a `Handler` instance to reconfigure logger while running
+    pub fn handle(&self) -> Handle {
+        Handle {
+            shared: self.0.clone(),
+        }
+    }
 }
 
 impl log::Log for Logger {
@@ -462,6 +468,11 @@ impl Handle {
         log::set_max_level(shared.root.max_log_level());
         self.shared.store(Arc::new(shared));
     }
+
+    /// Get the maximum log level according to the current configuration
+    pub fn max_log_level(&self) -> LevelFilter {
+        self.shared.load().root.max_log_level()
+    }
 }
 
 #[cfg(test)]