Skip to content

Commit

Permalink
Move logging into an independent crate
Browse files Browse the repository at this point in the history
  • Loading branch information
danielsanchezq committed Aug 3, 2020
1 parent 526c674 commit e68b75c
Show file tree
Hide file tree
Showing 11 changed files with 70 additions and 18 deletions.
1 change: 1 addition & 0 deletions Cargo.toml
@@ -1,5 +1,6 @@
[workspace]
members = [
"logging-lib",
"vit-servicing-station-cli",
"vit-servicing-station-lib",
"vit-servicing-station-server",
Expand Down
14 changes: 14 additions & 0 deletions logging-lib/Cargo.toml
@@ -0,0 +1,14 @@
[package]
name = "logging-lib"
version = "0.1.0"
authors = ["danielsanchezq <daniel.sanchez@iohk.io>"]
edition = "2018"

# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html

[dependencies]
chrono = { version = "0.4.13", features = ["serde"] }
log = "0.4.11"
serde = "1.0.114"
serde_json = "1.0.57"
simplelog = "0.8.0"
File renamed without changes.
3 changes: 3 additions & 0 deletions logging-lib/src/lib.rs
@@ -0,0 +1,3 @@
pub mod config;
pub mod messages;
mod methods;
File renamed without changes.
43 changes: 43 additions & 0 deletions logging-lib/src/methods.rs
@@ -0,0 +1,43 @@
#[macro_export]
macro_rules! log {
($lvl:expr, $($arg:tt)+) => (
let formatted_message = format_args!($($arg)+).to_string();
let level = $lvl;
$crate::messages::LogMessageBuilder::default().with_level(level).with_message(formatted_message).build().log();
)
}

#[macro_export]
macro_rules! error {
($($arg:tt)+) => (
$crate::log!(log::Level::Error, $($arg)+)
)
}

#[macro_export]
macro_rules! warn {
($($arg:tt)+) => (
$crate::log!(log::Level::Warn, $($arg)+)
)
}

#[macro_export]
macro_rules! info {
($($arg:tt)+) => (
$crate::log!(log::Level::Info, $($arg)+)
)
}

#[macro_export]
macro_rules! debug {
($($arg:tt)+) => (
$crate::log!(log::Level::Debug, $($arg)+)
)
}

#[macro_export]
macro_rules! trace {
($($arg:tt)+) => (
$crate::log!($log::Level::Trace, $($arg)+)
)
}
1 change: 0 additions & 1 deletion vit-servicing-station-lib/src/lib.rs
Expand Up @@ -7,7 +7,6 @@ extern crate structopt;
extern crate diesel_migrations;

pub mod db;
pub mod logging;
pub mod server;
pub mod utils;
pub mod v0;
5 changes: 0 additions & 5 deletions vit-servicing-station-lib/src/logging/methods.rs

This file was deleted.

6 changes: 0 additions & 6 deletions vit-servicing-station-lib/src/logging/mod.rs

This file was deleted.

1 change: 1 addition & 0 deletions vit-servicing-station-server/Cargo.toml
Expand Up @@ -8,6 +8,7 @@ edition = "2018"

[dependencies]
log = "0.4.11"
logging-lib = { path = "../logging-lib" }
structopt = "0.3.14"
tokio = { version = "0.2.21", features = ["macros"] }
vit-servicing-station-lib = { path = "../vit-servicing-station-lib" }
14 changes: 8 additions & 6 deletions vit-servicing-station-server/src/main.rs
@@ -1,9 +1,12 @@
use structopt::StructOpt;

use vit_servicing_station_lib::{
db, logging::config::config_log, server, server::exit_codes::ApplicationExitCode,
server::settings as server_settings, server::settings::ServiceSettings, v0,
db, server, server::exit_codes::ApplicationExitCode, server::settings as server_settings,
server::settings::ServiceSettings, v0,
};

use logging_lib::{config::config_log, *};

#[tokio::main]
async fn main() {
// load settings from command line (defaults to env variables)
Expand All @@ -13,7 +16,7 @@ async fn main() {
if let Some(settings_file) = &settings.in_settings_file {
let in_file_settings = server_settings::load_settings_from_file(settings_file)
.unwrap_or_else(|e| {
log::error!("Error loading settings from file {}, {}", settings_file, e);
error!("Error loading settings from file {}, {}", settings_file, e);
std::process::exit(ApplicationExitCode::LoadSettingsError.into())
});
// merge input file settings override by cli arguments
Expand Down Expand Up @@ -56,10 +59,9 @@ async fn main() {

let app = v0::filter(context).await;

log::info!(
info!(
"Running server at {}, database located at {}",
settings.address,
settings.db_url
settings.address, settings.db_url
);

// run server with settings
Expand Down

0 comments on commit e68b75c

Please sign in to comment.