snowykami/meilisearch
1
0
Fork 0
mirror of https://github.com/meilisearch/meilisearch.git synced 2025-09-06 20:56:31 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix reserved name check

Browse Source
This commit is contained in:
Mubelotix
2025-07-30 15:44:51 +02:00
parent b565ec1497
commit e88480c7c4
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
crates/meilisearch/src/routes/webhooks.rs
View File

@ -189,10 +189,6 @@ async fn patch_webhooks(
old_webhook: Option<Webhook>, old_webhook: Option<Webhook>,
new_webhook: WebhookSettings, new_webhook: WebhookSettings,
) -> Result<Webhook, WebhooksError> { ) -> Result<Webhook, WebhooksError> {
if name.starts_with('_') {
return Err(WebhooksError::ReservedWebhook(name.to_owned()));
}
let (old_url, mut headers) = let (old_url, mut headers) =
old_webhook.map(|w| (Some(w.url), w.headers)).unwrap_or((None, BTreeMap::new())); 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 { match new_webhooks {
Setting::Set(new_webhooks) => { Setting::Set(new_webhooks) => {
for (name, new_webhook) in new_webhooks { for (name, new_webhook) in new_webhooks {
if name.starts_with('_') {
return Err(WebhooksError::ReservedWebhook(name).into());
}
match new_webhook { match new_webhook {
Setting::Set(new_webhook) => { Setting::Set(new_webhook) => {
let old_webhook = webhooks.remove(&name); let old_webhook = webhooks.remove(&name);