DataDog / datadogpy

The Datadog Python library
https://datadoghq.com/
Other
612 stars 302 forks source link

Support IPv6 hostnames in `dogstatsd` #784

Open phil-scale opened 1 year ago

phil-scale commented 1 year ago

Note: If you have a feature request, you should contact support so the request can be properly tracked.

Is your feature request related to a problem? Please describe.

Support IPv6 hostnames in the DogStatsd class so that we can resolve the following issue in IPv6-addressed environments

>>> from datadog.dogstatsd.base import DogStatsd
>>> DogStatsd._get_udp_socket("::", 8126)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/opt/miniconda3/envs/hmi/lib/python3.8/site-packages/datadog/dogstatsd/base.py", line 544, in _get_udp_socket
    sock.connect((host, port))
socket.gaierror: [Errno 8] nodename nor servname provided, or not known

Describe the solution you'd like Proposed fix: https://github.com/DataDog/datadogpy/pull/783

Describe alternatives you've considered A clear and concise description of any alternative solutions or features you've considered.

Additional context Add any other context or screenshots about the feature request here.

github-actions[bot] commented 1 year ago

Thanks for your contribution!

This issue has been automatically marked as stale because it has not had activity in the last 30 days. Note that the issue will not be automatically closed, but this notification will remind us to investigate why there's been inactivity. Thank you for participating in the Datadog open source community.

If you would like this issue to remain open:

  1. Verify that you can still reproduce the issue in the latest version of this project.

  2. Comment that the issue is still reproducible and include updated details requested in the issue template.