session: do not assign conns to nil on Close()

Let the runtime handle garbage collection, and avoid errors where
methods like Unsubscribe() are called after Close().

vici/events.go

@@ -74,12 +74,7 @@ func newEventListener(cc *clientConn) *eventListener {
 
 // Close closes the event channel.
 func (el *eventListener) Close() error {
-	if el.cc != nil {
-		el.cc.Close()
-		el.cc = nil
-	}
-
-	return nil
+	return el.cc.Close()
 }
 
 // listen is responsible for receiving all packets from the daemon. This includes

vici/session.go

@@ -102,21 +102,12 @@ func (s *Session) newClientConn() (*clientConn, error) {
 
 // Close closes the vici session.
 func (s *Session) Close() error {
-	if err := s.el.Close(); err != nil {
-		return err
-	}
+	err := e.el.Close()
 
 	s.mu.Lock()
 	defer s.mu.Unlock()
-	if s.cc != nil {
-		if err := s.cc.Close(); err != nil {
-			return err
-		}
-
-		s.cc = nil
-	}
 
-	return nil
+	return errors.Join(err, s.cc.Close())
 }
 
 // SessionOption is used to specify additional options