influxdb-python icon indicating copy to clipboard operation
influxdb-python copied to clipboard

Published 20 hours ago • verified publisher icon influxdata

chore: InfluxDBClient.close should also close udp socket

Open yabberyabber opened this issue 3 years ago • 0 comments

Currently, InfluxDBClient.close only works for http connections. The function explicitly skips closing the udp_socket member. UDP connections may not cost the system anything but leaving any sort of socket open can cause certain dynamic analysis tools to complain.

With this patch we explicitly close the socket which quiets our linter's warnings.

Contributor checklist
  • [ ] Builds are passing
  • [ ] New tests have been added (for feature additions)

yabberyabber avatar Nov 23 '21 22:11 yabberyabber