mirrors/postgsail
1
0
Fork 0
mirror of https://github.com/xbgmsharp/postgsail.git synced 2025-09-17 19:27:49 +00:00
Code Issues Packages Projects Releases Wiki Activity

Update logging for api.metrics trigger.

Browse Source 
Force vessel_id to import from SQL cli run as username role
This commit is contained in:
xbgmsharp
2023-06-26 13:31:44 +02:00
parent 894dbf0667
commit 3eb67abedb
1 changed files with 13 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
initdb/02_1_signalk_api.sql
View File

@@ -402,16 +402,20 @@ CREATE FUNCTION metrics_trigger_fn() RETURNS trigger AS $metrics$
valid_status BOOLEAN; valid_status BOOLEAN;
_vessel_id TEXT; _vessel_id TEXT;
BEGIN BEGIN
-- Force vessel_id to import from SQL cli -- Force vessel_id to import from SQL cli run as username role
SELECT vessel_id INTO _vessel_id FROM api.metadata WHERE client_id = NEW.client_id; SELECT vessel_id INTO _vessel_id FROM api.metadata WHERE client_id = NEW.client_id;
IF _vessel_id IS NOT NULL THEN IF _vessel_id IS NOT NULL THEN
NEW.vessel_id = _vessel_id; NEW.vessel_id = _vessel_id;
PERFORM set_config('vessel.id', _vessel_id, false) as vessel_id; PERFORM set_config('vessel.id', _vessel_id, false) as vessel_id;
--RAISE NOTICE 'metrics_trigger_fn set vessel_id [%] [%] ', NEW.vessel_id, NEW.client_id; RAISE NOTICE 'metrics_trigger_fn set vessel_id [%] [%] ', NEW.vessel_id, NEW.client_id;
ELSE
RAISE EXCEPTION 'Invalid vessel.id'
USING HINT = 'Unknow vessel.id';
RETURN NULL; -- Ingore insert if no vessel_id
END IF; END IF;
-- Set client_id to new value to allow RLS -- Set client_id to new value to allow RLS
--PERFORM set_config('vessel.client_id', NEW.client_id, false); --PERFORM set_config('vessel.client_id', NEW.client_id, false);
NEW.vessel_id = current_setting('vessel.id'); NEW.vessel_id = current_setting('vessel.id', true);
--RAISE NOTICE 'metrics_trigger_fn client_id [%]', NEW.client_id; --RAISE NOTICE 'metrics_trigger_fn client_id [%]', NEW.client_id;
-- Boat metadata are check using api.metrics REFERENCES to api.metadata -- Boat metadata are check using api.metrics REFERENCES to api.metadata
-- Fetch the latest entry to compare status against the new status to be insert -- Fetch the latest entry to compare status against the new status to be insert
@@ -423,31 +427,31 @@ CREATE FUNCTION metrics_trigger_fn() RETURNS trigger AS $metrics$
--RAISE NOTICE 'Metrics Status, New:[%] Previous:[%]', NEW.status, previous_status; --RAISE NOTICE 'Metrics Status, New:[%] Previous:[%]', NEW.status, previous_status;
IF previous_time = NEW.time THEN IF previous_time = NEW.time THEN
-- Ignore entry if same time -- Ignore entry if same time
RAISE WARNING 'Metrics Ignoring metric, duplicate time [%] = [%]', previous_time, NEW.time; RAISE WARNING 'Metrics Ignoring metric, vessel_id [%], duplicate time [%] = [%]', NEW.vessel_id, previous_time, NEW.time;
RETURN NULL; RETURN NULL;
END IF; END IF;
IF previous_time > NEW.time THEN IF previous_time > NEW.time THEN
-- Ignore entry if new time is later than previous time -- Ignore entry if new time is later than previous time
RAISE WARNING 'Metrics Ignoring metric, new time is older [%] > [%]', previous_time, NEW.time; RAISE WARNING 'Metrics Ignoring metric, vessel_id [%], new time is older [%] > [%]', NEW.vessel_id, previous_time, NEW.time;
RETURN NULL; RETURN NULL;
END IF; END IF;
-- Check if latitude or longitude are null -- Check if latitude or longitude are null
IF NEW.latitude IS NULL OR NEW.longitude IS NULL THEN IF NEW.latitude IS NULL OR NEW.longitude IS NULL THEN
-- Ignore entry if null latitude,longitude -- Ignore entry if null latitude,longitude
RAISE WARNING 'Metrics Ignoring metric, null latitude,longitude [%] [%]', NEW.latitude, NEW.longitude; RAISE WARNING 'Metrics Ignoring metric, vessel_id [%], null latitude,longitude [%] [%]', NEW.vessel_id, NEW.latitude, NEW.longitude;
RETURN NULL; RETURN NULL;
END IF; END IF;
-- Check if status is null -- Check if status is null
IF NEW.status IS NULL THEN IF NEW.status IS NULL THEN
RAISE WARNING 'Metrics Unknow NEW.status from vessel [%], set to default moored', NEW; RAISE WARNING 'Metrics Unknow NEW.status, null status, set to default moored [%]', NEW;
NEW.status := 'moored'; NEW.status := 'moored';
END IF; END IF;
IF previous_status IS NULL THEN IF previous_status IS NULL THEN
IF NEW.status = 'anchored' THEN IF NEW.status = 'anchored' THEN
RAISE WARNING 'Metrics Unknow previous_status from vessel [%], set to default current status [%]', previous_status, NEW.status; RAISE WARNING 'Metrics Unknow previous_status from vessel_id [%], [%] set to default current status [%]', NEW.vessel_id, previous_status, NEW.status;
previous_status := NEW.status; previous_status := NEW.status;
ELSE ELSE
RAISE WARNING 'Metrics Unknow previous_status from vessel [%], set to default status moored vs [%]', previous_status, NEW.status; RAISE WARNING 'Metrics Unknow previous_status from vessel_id [%], [%] set to default status moored vs [%]', NEW.vessel_id, previous_status, NEW.status;
previous_status := 'moored'; previous_status := 'moored';
END IF; END IF;
-- Add new stay as no previous entry exist -- Add new stay as no previous entry exist