fix: resolve clippy warnings and missing redis_key_separator field
This commit is contained in:
parent
a5e1a64247
commit
20622bc762
|
|
@ -264,7 +264,7 @@ fn translate_order_by(items: &[OrderByExpr]) -> Result<Option<Json>, String> {
|
|||
fn extract_limit_offset(query: &Query) -> Result<(Option<usize>, Option<usize>), String> {
|
||||
let limit = match &query.limit_clause {
|
||||
Some(sqlparser::ast::LimitClause::LimitOffset { limit, .. }) => {
|
||||
limit.as_ref().map(|expr| usize_literal(expr)).transpose()?
|
||||
limit.as_ref().map(usize_literal).transpose()?
|
||||
}
|
||||
_ => None,
|
||||
};
|
||||
|
|
|
|||
|
|
@ -169,7 +169,7 @@ fn is_safe_read_pragma(upper_stripped: &str) -> bool {
|
|||
let pragma_name = &rest[..name_end];
|
||||
|
||||
// Check if it's in the safe list
|
||||
if !SAFE_READ_PRAGMA_NAMES.iter().any(|&safe| pragma_name == safe) {
|
||||
if !SAFE_READ_PRAGMA_NAMES.contains(&pragma_name) {
|
||||
return false;
|
||||
}
|
||||
|
||||
|
|
|
|||
|
|
@ -190,6 +190,7 @@ mod tests {
|
|||
redis_sentinel_password: String::new(),
|
||||
redis_sentinel_tls: false,
|
||||
redis_cluster_nodes: String::new(),
|
||||
redis_key_separator: dbx_core::models::connection::default_redis_key_separator(),
|
||||
etcd_endpoints: String::new(),
|
||||
external_config: None,
|
||||
jdbc_driver_class: None,
|
||||
|
|
|
|||
Loading…
Reference in New Issue