cybertec-postgresql / pgwatch / 15214639834

74%

Build Type

push

github

Committed by web-flow

Commit Message

[-] fix `QueryMeasurements` nil pointer dereference, fixes #766 (#767)

* keep existing connection in SourceConns.SyncFromReader()

Since `SourceConn` passed to reapMetricMeasurements() it's crucial
to keep the existing source with connection alive, so the reaper
keeps it's work. And there is no need to close the connection because
it might be used right now in some metric fetcher. Let Reaper to cancel
all disabled monitored sources in the main loop of Reap() function.

* fix tests

Run Details

4 of 4 new or added lines in 2 files covered. (100.0%)

1 existing line in 1 file now uncovered.

2444 of 4431 relevant lines covered (55.16%)

0.63 hits per line