gatewayd-io / gatewayd / 22079022632

59%

Build Type

push

github

Committed by web-flow

Commit Message

Pre-authenticate backend pool connections with PostgreSQL startup handshake (#724)

* Pre-authenticate the connections in the available pool
* Drain both OnTraffic goroutines before connection recycling
Rewrite OnTraffic to use sync.WaitGroup so both the client->server and
server->client goroutines are fully drained before returning. When one
goroutine exits it expires the other's read deadline, ensuring prompt
teardown. After both exit the backend deadline is cleared so Disconnect
can run DISCARD ALL on an idle connection. This fixes the "unexpected
EOF" errors during session reset and eliminates a goroutine leak where
the second goroutine would block forever on the stopConnection channel.
Also snapshot net.Conn in Client.Send/Receive under the mutex to prevent
a nil-pointer panic when Proxy.Shutdown closes the client concurrently.
* Update deps
* Update stubs
* Format
* Revert config
* Update deps
* Update docs
* Fix linter issues
* Remove unused variable
* Fix TextParser panic with prometheus/common v0.67.5
Use expfmt.NewTextParser(model.UTF8Validation) instead of the bare
struct literal, which defaults to UnsetValidation and panics.
* Fix connection counter race in server shutdown
Replace the two-goroutine stopConnection channel pattern with a single
goroutine that decrements connections after OnTraffic returns, eliminating
the race where select could pick stopServer over stopCh.
* Fix port conflict
* Silence noisy raft transport errors in logs
Replace os.Stderr with io.Discard for the Raft TCP transport logger
to suppress expected connection reset errors that occur during normal
peer removal operations.

Run Details

202 of 343 new or added lines in 7 files covered. (58.89%)

56 existing lines in 2 files now uncovered.

5738 of 9689 relevant lines covered (59.22%)

479.3 hits per line