Fix error handling in pg-sidecar (chef#641)

The defer Close was happening too soon. We need to check if we
have an error first, because we get nil back if there is an error.

Also, NewClient was leaking a database connection if an error occured
which I fixed.

Signed-off-by: Jay Mundrawala <jmundrawala@chef.io>

Fix error handling in pg-sidecar (chef#641)

The defer Close was happening too soon. We need to check if we
have an error first, because we get nil back if there is an error.

Also, NewClient was leaking a database connection if an error occured
which I fixed.

Signed-off-by: Jay Mundrawala <jmundrawala@chef.io>

Fix error handling in pg-sidecar (chef#641)

The defer Close was happening too soon. We need to check if we
have an error first, because we get nil back if there is an error.

Also, NewClient was leaking a database connection if an error occured
which I fixed.

Signed-off-by: Jay Mundrawala <jmundrawala@chef.io>

Fix error handling in pg-sidecar (chef#641)

The defer Close was happening too soon. We need to check if we
have an error first, because we get nil back if there is an error.

Also, NewClient was leaking a database connection if an error occured
which I fixed.

Signed-off-by: Jay Mundrawala <jmundrawala@chef.io>

Fix error handling in pg-sidecar (chef#641)

The defer Close was happening too soon. We need to check if we
have an error first, because we get nil back if there is an error.

Also, NewClient was leaking a database connection if an error occured
which I fixed.

Signed-off-by: Jay Mundrawala <jmundrawala@chef.io>

[A2-879] Refactor DeleteRule to work with staged rules (chef#642)

Fix error handling in pg-sidecar (chef#641)

The defer Close was happening too soon. We need to check if we
have an error first, because we get nil back if there is an error.

Also, NewClient was leaking a database connection if an error occured
which I fixed.

Signed-off-by: Jay Mundrawala <jmundrawala@chef.io>

[A2-879] Refactor DeleteRule to work with staged rules (chef#642)

[A2-879] Refactor DeleteRule to work with staged rules (chef#642)

Fix error handling in pg-sidecar (chef#641)

The defer Close was happening too soon. We need to check if we
have an error first, because we get nil back if there is an error.

Also, NewClient was leaking a database connection if an error occured
which I fixed.

Signed-off-by: Jay Mundrawala <jmundrawala@chef.io>