Skip to content
This repository was archived by the owner on Jul 11, 2021. It is now read-only.

Commit 5824666

Browse files
author
Simone Mosciatti
committed
add logging and remove escape from queries
1 parent fe7e36f commit 5824666

File tree

4 files changed

+14
-1
lines changed

4 files changed

+14
-1
lines changed

Cargo.lock

Lines changed: 1 addition & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

Cargo.toml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@ libc = "0.2.21"
1717
fnv = "1.0.5"
1818
uuid = { version = "0.4", features = ["v4"] }
1919
log = "0.3.8"
20+
env_logger = "0.4.3"
2021

2122
[features]
2223
default = []

src/lib.rs

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,9 @@ extern crate uuid;
33

44
#[macro_use]
55
extern crate log;
6+
extern crate env_logger;
7+
8+
use env_logger::{LogBuilder, LogTarget};
69

710
use std::ffi::{CString, CStr};
811
use std::mem;
@@ -681,6 +684,12 @@ pub extern "C" fn RedisModule_OnLoad(
681684

682685
sql::disable_global_memory_statistics();
683686

687+
LogBuilder::new()
688+
.filter(None, log::LogLevelFilter::Debug)
689+
.target(LogTarget::Stdout)
690+
.init()
691+
.unwrap();
692+
684693
let c_data_type_name = CString::new("rediSQLDB").unwrap();
685694
let ptr_data_type_name = c_data_type_name.as_ptr();
686695

src/redis.rs

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -172,7 +172,9 @@ fn parse_args(argv: *mut *mut ffi::RedisModuleString,
172172
let mut args: Vec<String> = Vec::with_capacity(argc as usize);
173173
for i in 0..argc {
174174
let redis_str = unsafe { *argv.offset(i as isize) };
175-
args.push(string_ptr_len(redis_str));
175+
let mut arg = string_ptr_len(redis_str);
176+
arg = arg.replace("\\", "");
177+
args.push(arg);
176178
}
177179
Ok(args)
178180
}

0 commit comments

Comments
 (0)