Skip to content

Commit 4a305d8

Browse files
Fix error on webhook parse failure
1 parent 247d419 commit 4a305d8

File tree

1 file changed

+8
-1
lines changed

1 file changed

+8
-1
lines changed

src/github/webhooks.rs

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,14 @@ impl FromDataSimple for Event {
5454
let payload = match parse_event(event_name, &body) {
5555
Ok(p) => p,
5656
Err(DashError::Serde(why)) => {
57-
info!("failed to parse webhook payload: {:?}", why);
57+
error!("failed to parse webhook payload: {:?}", why);
58+
return Failure((
59+
Status::BadRequest,
60+
"failed to deserialize request payload",
61+
));
62+
}
63+
Err(DashError::SerdePath(why)) => {
64+
error!("failed to parse webhook payload: {:?}", why);
5865
return Failure((
5966
Status::BadRequest,
6067
"failed to deserialize request payload",

0 commit comments

Comments
 (0)