Skip to content

[mobile] Skip localhost subdomain with trailing dot test on Android#127862

Open
github-actions[bot] wants to merge 2 commits intomainfrom
mobile-add-dns-trailingdot-skip-9117a239b194f376
Open

[mobile] Skip localhost subdomain with trailing dot test on Android#127862
github-actions[bot] wants to merge 2 commits intomainfrom
mobile-add-dns-trailingdot-skip-9117a239b194f376

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot commented May 6, 2026

Reasoning

The test DnsGetHostEntry_LocalhostSubdomainWithTrailingDot_ReturnsLoopback expects DNS queries for *.localhost. (with trailing dot) to resolve to loopback addresses, following RFC 6761. However, on Android devices, these queries resolve to actual network addresses (both IPv4 and IPv6) instead of loopback.

This is the same root cause as DnsGetHostEntry_LocalhostSubdomain_ReturnsLoopback (without trailing dot), which is already disabled on Android via ActiveIssue #126456. Android's DNS resolver does not treat .localhost subdomains as reserved names that must resolve to loopback, regardless of whether a trailing dot is present.

Adding the same [ActiveIssue] attribute to this test variant ensures consistent behavior across all localhost subdomain test cases on Android.

Impact on platforms

Failing on Android in build 1406427 (2026-05-03):

  • android-arm64 / Windows.11.Amd64.Android.Open / exit code 1
  • android-arm / Windows.11.Amd64.Android.Open / exit code 1
  • android-x64 / Windows.11.Amd64.Android.Open / exit code 1
  • android-x86 / Windows.11.Amd64.Android.Open / exit code 1

Errors log

From android-arm64 Helix work item System.Net.NameResolution.Functional.Tests (console log):

[FAIL] System.Net.NameResolution.Tests.GetHostEntryTest.DnsGetHostEntry_LocalhostSubdomainWithTrailingDot_ReturnsLoopback(hostName: "foo.localhost.")
Assert.All() Failure: 11 out of 12 items in the collection did not pass.
[1]:  Item:  fe80::e0ef:4bff:feb4:bb16%47
      Error: Expected loopback address but got: fe80::e0ef:4bff:feb4:bb16%47
[2]:  Item:  2001:4898:502:3:e0ef:4bff:feb4:bb16
      Error: Expected loopback address but got: 2001:4898:502:3:e0ef:4bff:feb4:bb16
[11]: Item:  10.64.36.83
      Error: Expected loopback address but got: 10.64.36.83

First build it occurred

First observed (within the scanned window) in build 1406427, finished on 2026-05-03T09:02:27Z.

Note: This test failure has the same root cause as existing issue #126456, which has been present since April 2026. The "first occurrence" reflects only the scanned window, not the true origin.

Note

This change was generated by the Mobile Platform Failure Scanner workflow.

Generated by Mobile Platform Failure Scanner · ● 6.8M ·

The test DnsGetHostEntry_LocalhostSubdomainWithTrailingDot_ReturnsLoopback
expects DNS queries for `*.localhost.` to resolve to loopback, but Android
devices resolve these to actual network addresses.

Related to existing ActiveIssue for DnsGetHostEntry_LocalhostSubdomain_ReturnsLoopback
which has the same root cause.

Fixes #126456

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@dotnet-policy-service
Copy link
Copy Markdown
Contributor

Tagging subscribers to this area: @karelz, @dotnet/ncl
See info in area-owners.md if you want to be subscribed.

@kotlarmilos kotlarmilos marked this pull request as ready for review May 6, 2026 11:47
Copilot AI review requested due to automatic review settings May 6, 2026 11:47
@kotlarmilos kotlarmilos added this to the 11.0.0 milestone May 6, 2026
@kotlarmilos kotlarmilos requested review from rzikm and simonrozsival May 6, 2026 11:47
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR addresses a platform-specific test failure in System.Net.NameResolution functional tests by skipping an RFC 6761 localhost-subdomain test case on Android, where the platform DNS resolver does not consistently return loopback addresses for *.localhost. names.

Changes:

  • Mark DnsGetHostEntry_LocalhostSubdomainWithTrailingDot_ReturnsLoopback as an active issue on Android using existing issue #126456.
Show a summary per file
File Description
src/libraries/System.Net.NameResolution/tests/FunctionalTests/GetHostEntryTest.cs Adds an [ActiveIssue] skip on Android for the trailing-dot localhost subdomain loopback assertion test.

Copilot's findings

  • Files reviewed: 1/1 changed files
  • Comments generated: 0

Copy link
Copy Markdown
Member

@rzikm rzikm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants