Skip to content

fix: make logging safer#1008

Open
0xwitty wants to merge 1 commit into
zkonduit:mainfrom
0xwitty:patch-2
Open

fix: make logging safer#1008
0xwitty wants to merge 1 commit into
zkonduit:mainfrom
0xwitty:patch-2

Conversation

@0xwitty
Copy link
Copy Markdown

@0xwitty 0xwitty commented Sep 2, 2025

spotted a few things that could crash the app:

  • switched record.args().as_str().unwrap()&format!("{}", record.args()) so logs work even if arguments aren’t strings.
  • tweaked FormatterFormatter<'_> for the right type in the format function.
  • replaced unwrap() on RUST_LOG with if let Ok(val) = env::var("RUST_LOG") to avoid panicking when it’s not set.

now logging should be smoother and safer :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant