fix: deleting a nonexistent key gets a FAIL

This commit is contained in:
Shivaram Lingamneni 2025-06-22 18:27:03 -04:00
parent 9c4c337393
commit fc388573ae

View file

@ -3200,9 +3200,10 @@ func metadataRegisteredHandler(client *Client, config *Config, subcommand string
} else { } else {
if updated := targetObj.DeleteMetadata(key); updated { if updated := targetObj.DeleteMetadata(key); updated {
notifySubscribers(server, rb.session, targetObj, target, key, "", false) notifySubscribers(server, rb.session, targetObj, target, key, "", false)
rb.Add(nil, server.name, RPL_KEYNOTSET, client.Nick(), target, key, client.t("Key deleted"))
} else {
rb.Add(nil, server.name, "FAIL", "METADATA", "KEY_NOT_SET", utils.SafeErrorParam(key), client.t("Metadata key not set"))
} }
// acknowledge to the client whether or not there was a real update
rb.Add(nil, server.name, RPL_KEYNOTSET, client.Nick(), target, key, client.t("Key deleted"))
} }
case "get": case "get":