From 3e6a0f9e63339978ba8741ad897de38cd2493a2a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ask=20Bj=C3=B8rn=20Hansen?= Date: Sun, 3 Aug 2025 14:53:21 -0700 Subject: [PATCH] fix(api): include deleted monitors in name-based lookups Remove status filter from GetMonitorByNameAndIPVersion query to allow historical score data for deleted monitors to be accessible when querying by monitor name/TLS name, making behavior consistent with ID-based queries. --- ntpdb/query.sql.go | 3 +-- query.sql | 3 +-- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/ntpdb/query.sql.go b/ntpdb/query.sql.go index a57cd39..a6efc9d 100644 --- a/ntpdb/query.sql.go +++ b/ntpdb/query.sql.go @@ -17,8 +17,7 @@ select id, id_token, type, user_id, account_id, hostname, location, ip, ip_versi where tls_name like ? AND (ip_version = ? OR (type = 'score' AND ip_version IS NULL)) AND - is_current = 1 AND - status != 'deleted' + is_current = 1 order by id limit 1 ` diff --git a/query.sql b/query.sql index f05dd76..9ab98b2 100644 --- a/query.sql +++ b/query.sql @@ -52,8 +52,7 @@ select * from monitors where tls_name like sqlc.arg('tls_name') AND (ip_version = sqlc.arg('ip_version') OR (type = 'score' AND ip_version IS NULL)) AND - is_current = 1 AND - status != 'deleted' + is_current = 1 order by id limit 1;