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.
This commit is contained in:
@@ -17,8 +17,7 @@ select id, id_token, type, user_id, account_id, hostname, location, ip, ip_versi
|
|||||||
where
|
where
|
||||||
tls_name like ? AND
|
tls_name like ? AND
|
||||||
(ip_version = ? OR (type = 'score' AND ip_version IS NULL)) AND
|
(ip_version = ? OR (type = 'score' AND ip_version IS NULL)) AND
|
||||||
is_current = 1 AND
|
is_current = 1
|
||||||
status != 'deleted'
|
|
||||||
order by id
|
order by id
|
||||||
limit 1
|
limit 1
|
||||||
`
|
`
|
||||||
|
|||||||
@@ -52,8 +52,7 @@ select * from monitors
|
|||||||
where
|
where
|
||||||
tls_name like sqlc.arg('tls_name') AND
|
tls_name like sqlc.arg('tls_name') AND
|
||||||
(ip_version = sqlc.arg('ip_version') OR (type = 'score' AND ip_version IS NULL)) AND
|
(ip_version = sqlc.arg('ip_version') OR (type = 'score' AND ip_version IS NULL)) AND
|
||||||
is_current = 1 AND
|
is_current = 1
|
||||||
status != 'deleted'
|
|
||||||
order by id
|
order by id
|
||||||
limit 1;
|
limit 1;
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user