Redesign page + Webhook notifications #43

Merged
ldr merged 16 commits from facelift into main 2023-12-25 21:23:53 +01:00
Showing only changes of commit 1406682b9c - Show all commits

Log warning about error during handlebars rendering of webhook message

Lukas Drescher 2023-12-25 19:06:12 +01:00

View file

@ -3,7 +3,7 @@ use std::sync::Arc;
use actix_tls::connect::rustls_0_20::reexports::ClientConfig; use actix_tls::connect::rustls_0_20::reexports::ClientConfig;
use awc::{Client, Connector}; use awc::{Client, Connector};
use handlebars::{Handlebars, RenderError}; use handlebars::{Handlebars, RenderError};
use log::debug; use log::{debug, error, warn};
use serde_json::json; use serde_json::json;
use url::Url; use url::Url;
@ -21,7 +21,15 @@ pub(crate) async fn send_new_offer_message(
webhook_url: Url, webhook_url: Url,
data: &NewOfferWebhookData, data: &NewOfferWebhookData,
) -> Result<bool, RenderError> { ) -> Result<bool, RenderError> {
let message_text = hb.render(template::WEBHOOK_NEW_OFFER_MESSAGE, data)?; let message_text = hb
.render(template::WEBHOOK_NEW_OFFER_MESSAGE, data)
.map_err(|err| {
warn!(
"Rendering webhook message failed with the following error: {}",
err
);
err
})?;
let msg = json!({ let msg = json!({
"content": message_text, "content": message_text,
@ -51,7 +59,7 @@ pub(crate) async fn send_new_offer_message(
true true
} }
Err(err) => { Err(err) => {
debug!("Webhook request failed with error: {}", err); warn!("Webhook request failed with error: {}", err);
false false
} }
}; };