From e88480c7c4d3f37d98fa80004ac0515048ead489 Mon Sep 17 00:00:00 2001 From: Mubelotix Date: Wed, 30 Jul 2025 15:44:51 +0200 Subject: [PATCH] Fix reserved name check --- crates/meilisearch/src/routes/webhooks.rs | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/crates/meilisearch/src/routes/webhooks.rs b/crates/meilisearch/src/routes/webhooks.rs index 631dd822d..9cf57c585 100644 --- a/crates/meilisearch/src/routes/webhooks.rs +++ b/crates/meilisearch/src/routes/webhooks.rs @@ -189,10 +189,6 @@ async fn patch_webhooks( old_webhook: Option, new_webhook: WebhookSettings, ) -> Result { - if name.starts_with('_') { - return Err(WebhooksError::ReservedWebhook(name.to_owned())); - } - let (old_url, mut headers) = old_webhook.map(|w| (Some(w.url), w.headers)).unwrap_or((None, BTreeMap::new())); @@ -232,6 +228,10 @@ async fn patch_webhooks( match new_webhooks { Setting::Set(new_webhooks) => { for (name, new_webhook) in new_webhooks { + if name.starts_with('_') { + return Err(WebhooksError::ReservedWebhook(name).into()); + } + match new_webhook { Setting::Set(new_webhook) => { let old_webhook = webhooks.remove(&name);