Should VConnection::do_io_close clean up all memory

[scw00]

Currently Http1ClientSession is only cleaned while transaction_done is called in HttpSM::kill_this.
even if the do_io_close has been called .

256   /**
257     Indicate that the VConnection is no longer needed.
258 
259     Once the state machine has finished using this VConnection, it
260     must call this function to indicate that the VConnection can
261     be deallocated.  After a close has been called, the VConnection
262     and underlying processor must not send any more events related
263     to this VConnection to the state machine. Likeswise, the state
264     machine must not access the VConnection or any VIOs obtained
265     from it after calling this method.
266 
267     @param lerrno indicates where a close is a normal close or an
268       abort. The difference between a normal close and an abort
269       depends on the underlying type of the VConnection.
270 
271   */
272   virtual void do_io_close(int lerrno = -1) = 0;

Http2ClientSession has the same issue too.

[github-actions]

This issue has been automatically marked as stale because it has not had recent activity. Marking it stale to flag it for further consideration by the community.