close db properly

hugodutka · hugodutka · commit 471bd6d9ac12 · 2025-06-19T09:42:03.000Z
diff --git a/cli/server.go b/cli/server.go
@@ -2363,10 +2363,8 @@ func ConnectToPostgres(ctx context.Context, logger slog.Logger, driver string, d
 	if !version.Next() {
 		// it's critical we assign to the err variable, otherwise the defer statement
 		// that runs db.Close() will not execute it
-		if err = version.Err(); err != nil {
-			return nil, xerrors.Errorf("no rows returned for version select: %w", err)
-		}
-		return nil, xerrors.Errorf("no rows returned for version select")
+		err = xerrors.Errorf("no rows returned for version select: %w", version.Err())
+		return nil, err
 	}
 	var versionNum int
 	err = version.Scan(&versionNum)
