Skip to content

Skip DNS resolution when agent is bound to localhost#841

Merged
kevingosse merged 2 commits into
masterfrom
kevin/ipv6
Aug 14, 2020
Merged

Skip DNS resolution when agent is bound to localhost#841
kevingosse merged 2 commits into
masterfrom
kevin/ipv6

Conversation

@kevingosse

Copy link
Copy Markdown
Contributor

On .net core, when resolving localhost on a machine with ipv6 enabled, ::1 is considered first. Unfortunately, the agent is bound only to ipv4, so the initial connection timeouts and adds a 2 seconds delay.

Changing localhost to 127.0.0.1 should fix this in the most common case.

@kevingosse
kevingosse requested a review from a team as a code owner August 7, 2020 10:04
@kevingosse
kevingosse merged commit 7839843 into master Aug 14, 2020
@kevingosse
kevingosse deleted the kevin/ipv6 branch August 14, 2020 14:45
zacharycmontoya added a commit that referenced this pull request Aug 17, 2020
zacharycmontoya added a commit that referenced this pull request Aug 18, 2020
zacharycmontoya added a commit that referenced this pull request Aug 18, 2020
…864)

This reverts commit 7839843. The tests need to be updated to accommodate the DNS change.
kevingosse added a commit that referenced this pull request Aug 19, 2020
* Skip DNS resolution when agent is bound to localhost
kevingosse added a commit that referenced this pull request Aug 24, 2020
* Skip DNS resolution when agent is bound to localhost
kevingosse added a commit that referenced this pull request Aug 25, 2020
* Skip DNS resolution when agent is bound to localhost
@zacharycmontoya zacharycmontoya added this to the 1.19.2 milestone Aug 31, 2020
@zacharycmontoya zacharycmontoya added the area:tracer The core tracer library (Datadog.Trace, does not include OpenTracing, native code, or integrations) label Aug 31, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:tracer The core tracer library (Datadog.Trace, does not include OpenTracing, native code, or integrations)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants