kemitix/git-next
1
1
Fork 1
Code Issues 17 Pull requests 1 Projects 1 Releases 41 Packages 2 Wiki Activity Actions

fix: webhook secret doesn't need to be base64 encoded
All checks were successful
Rust / build (push) Successful in 1m29s
Details
ci/woodpecker/push/cron-docker-builder Pipeline was successful
Details
ci/woodpecker/push/push-next Pipeline was successful
Details
ci/woodpecker/push/tag-created Pipeline was successful
Details

Browse source 
Closes kemitix/git-next#118
This commit is contained in:
Paul Campbell 2024-07-29 07:51:09 +01:00
parent b89431b779
commit 691a733fc3
1 changed files with 3 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
crates/cli/src/server/actor/handlers/notify_user.rs
View file

@ -39,7 +39,9 @@ async fn send_webhook(
tracing::warn!("Invalid notification configuration (config) - can't sent notification"); tracing::warn!("Invalid notification configuration (config) - can't sent notification");
return; return;
}; };
let Ok(webhook) = Webhook::new(webhook_config.secret().expose_secret()) else { let Ok(webhook) =
Webhook::from_bytes(webhook_config.secret().expose_secret().as_bytes().into())
else {
tracing::warn!("Invalid notification configuration (signer) - can't sent notification"); tracing::warn!("Invalid notification configuration (signer) - can't sent notification");
return; return;
}; };