diff --git a/server/src/handlers/logstream.rs b/server/src/handlers/logstream.rs index 9f2ff9883..ced5afa3c 100644 --- a/server/src/handlers/logstream.rs +++ b/server/src/handlers/logstream.rs @@ -244,15 +244,13 @@ pub async fn put_alert(req: HttpRequest, body: web::Json) -> } .to_http(), }, - Err(e) => { - return response::ServerResponse { - msg: format!( - "failed to set alert configuration for log stream {} due to err: {}", - stream_name, e - ), - code: StatusCode::BAD_REQUEST, - } - .to_http(); + Err(e) => response::ServerResponse { + msg: format!( + "failed to set alert configuration for log stream {} due to err: {}", + stream_name, e + ), + code: StatusCode::BAD_REQUEST, } + .to_http(), } } diff --git a/server/src/validator.rs b/server/src/validator.rs index 04e7a4e4d..14b332427 100644 --- a/server/src/validator.rs +++ b/server/src/validator.rs @@ -28,13 +28,13 @@ const DENIED_NAMES: &[&str] = &[ "select", "from", "where", "group", "by", "order", "limit", "offset", "join", "and", ]; -#[derive(Default, Debug, Clone, PartialEq, Serialize, Deserialize)] +#[derive(Default, Debug, Clone, PartialEq, Eq, Serialize, Deserialize)] #[serde(rename_all = "camelCase")] pub struct Alerts { pub alerts: Vec, } -#[derive(Default, Debug, Clone, PartialEq, Serialize, Deserialize)] +#[derive(Default, Debug, Clone, PartialEq, Eq, Serialize, Deserialize)] #[serde(rename_all = "camelCase")] pub struct Alert { pub name: String, @@ -43,7 +43,7 @@ pub struct Alert { pub target: Vec, } -#[derive(Default, Debug, Clone, PartialEq, Serialize, Deserialize)] +#[derive(Default, Debug, Clone, PartialEq, Eq, Serialize, Deserialize)] #[serde(rename_all = "camelCase")] pub struct Rule { pub field: String, @@ -52,7 +52,7 @@ pub struct Rule { pub within: String, } -#[derive(Default, Debug, Clone, PartialEq, Serialize, Deserialize)] +#[derive(Default, Debug, Clone, PartialEq, Eq, Serialize, Deserialize)] #[serde(rename_all = "camelCase")] pub struct Target { pub name: String,