influxdata / influxdb-python

Python client for InfluxDB
MIT License
1.7k stars 522 forks source link

chore: InfluxDBClient.close should also close udp socket #906

Open yabberyabber opened 3 years ago

yabberyabber commented 3 years ago

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