fix session leak

[zjtaozjtw]

What type of PR is this?

• bug
• feature
• enhancement

What problem(s) does this PR solve?

Issue(s) number:

#268

Description:

How do you solve it?

当new session 后，及时更新当前使用的session引用

Special notes for your reviewer, ex. impact of this fix, design document, etc:

[CLAassistant]

All committers have signed the CLA.

[wey-gu]

Thanks!

[codecov-commenter]

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.01 🎉

Comparison is base (bf9fe21) 61.26% compared to head (5ac91a7) 61.28%.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #269      +/-   ##
==========================================
+ Coverage   61.26%   61.28%   +0.01%     
==========================================
  Files           9        9              
  Lines        2308     2309       +1     
==========================================
+ Hits         1414     1415       +1     
  Misses        749      749              
  Partials      145      145              

Impacted Files	Coverage Δ
session_pool.go	60.96% <100.00%> (+0.11%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

[HarrisChu]

this pr would simply fix the issue, but we process session in separate functions. (new session, update or push to list)
prefer refactor those functions...
@Aiee WDYT

[HarrisChu]

maybe we would quick fix first, and then enhance these code in future

[Aiee]

maybe we would quick fix first, and then enhance these code in future

Makes sense to me.

[Aiee]

Nice catch! Thanks for your contribution.