Skip to content

Commit 56488c9

Browse files
committed
De-duplicate zulip webhook logging
1 parent 0d23103 commit 56488c9

File tree

1 file changed

+0
-5
lines changed

1 file changed

+0
-5
lines changed

src/zulip.rs

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -119,9 +119,6 @@ pub async fn webhook(
119119
// We are mixing network errors and "logic" error (like clap errors)
120120
// so don't return a 500. Long term we should decouple those.
121121

122-
// Log the full error
123-
tracing::error!(?err);
124-
125122
// Reply with a 200 and reply only with outermost error
126123
Json(Response {
127124
content: err.to_string(),
@@ -158,8 +155,6 @@ async fn process_zulip_request(ctx: Arc<Context>, req: Request) -> anyhow::Resul
158155
anyhow::bail!("Invalid authorization.");
159156
}
160157

161-
log::trace!("zulip hook: {req:?}");
162-
163158
// Zulip commands are only available to users in the team database
164159
let gh_id = match ctx.team.zulip_to_github_id(req.message.sender_id).await {
165160
Ok(Some(gh_id)) => gh_id,

0 commit comments

Comments
 (0)