If requestTLS fails, a PostgresConnection will be leaked, which asserts.

[Lukasa]

A client requesting TLS will hit this branch of code:

postgres-nio/Sources/PostgresNIO/Connection/PostgresConnection+Connect.swift

Lines 23 to 33 in 2334eab

	}.flatMap { (conn: PostgresConnection) in
	if let tlsConfiguration = tlsConfiguration {
	return conn.requestTLS(
	using: tlsConfiguration,
	serverHostname: serverHostname,
	logger: logger
	).map { conn }
	} else {
	return eventLoop.makeSucceededFuture(conn)
	}
	}

If the requestTLS call fails, the returned future will be failed. This will send us down an error path, meaning the PostgresConnection will never make it to user code. However, the PostgresConnection is not closed on that path: nothing in here closes the connection. This triggers the assert in deinit:

postgres-nio/Sources/PostgresNIO/Connection/PostgresConnection.swift

Lines 33 to 35 in 2334eab

	deinit {
	assert(self.isClosed, "PostgresConnection deinitialized before being closed.")
	}

This should be handled more gracefully, by closing the connection on error.