Correct the consume queue name, and don't panic on deeserialize errorss
This commit is contained in:
parent
5e2c57790e
commit
5d24478710
|
@ -67,7 +67,7 @@ fn main() {
|
|||
cfg.github(),
|
||||
)),
|
||||
easyamqp::ConsumeConfig {
|
||||
queue: "build-inputs".to_owned(),
|
||||
queue: "build-results".to_owned(),
|
||||
consumer_tag: format!("{}-github-comment-poster", cfg.whoami()),
|
||||
no_local: false,
|
||||
no_ack: false,
|
||||
|
|
|
@ -31,11 +31,11 @@ impl worker::SimpleWorker for GitHubCommentPoster {
|
|||
return match serde_json::from_slice(body) {
|
||||
Ok(e) => Ok(e),
|
||||
Err(e) => {
|
||||
println!(
|
||||
"Failed to deserialize BuildResult: {:?}",
|
||||
String::from_utf8(body.clone())
|
||||
);
|
||||
panic!("{:?}", e);
|
||||
Err(format!(
|
||||
"Failed to deserialize BuildResult: {:?}, err: {:}",
|
||||
String::from_utf8_lossy(&body.clone()),
|
||||
e
|
||||
))
|
||||
}
|
||||
};
|
||||
}
|
||||
|
|
Loading…
Reference in a new issue