Skip to content

[HTTP] Re-enable GetAsync_UnicodeHostName_SuccessStatusCodeInResponse test#124596

Open
ManickaP wants to merge 1 commit intodotnet:mainfrom
ManickaP:fix/reenable-29424-unicode-hostname
Open

[HTTP] Re-enable GetAsync_UnicodeHostName_SuccessStatusCodeInResponse test#124596
ManickaP wants to merge 1 commit intodotnet:mainfrom
ManickaP:fix/reenable-29424-unicode-hostname

Conversation

@ManickaP
Copy link
Member

The external Korean Starbucks website (스타벅스코리아.com / xn--oy2b35ckwhba574atvuzkc.com) that was unreachable in 2019 is now accessible again. Remove the ActiveIssue attribute to re-enable the test, keeping the original test logic that exercises the full Unicode hostname to punycode IDN conversion path.

Excludes H/3 as the site doesn't support it.

Fixes #29424

The external Korean Starbucks website (스타벅스코리아.com /
xn--oy2b35ckwhba574atvuzkc.com) that was unreachable in 2019 is now
accessible again. Remove the ActiveIssue attribute to re-enable the
test, keeping the original test logic that exercises the full
Unicode hostname to punycode IDN conversion path.

Fixes dotnet#29424

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings February 19, 2026 14:22
@ManickaP
Copy link
Member Author

/azp run runtime-libraries-coreclr outerloop

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@dotnet-policy-service
Copy link
Contributor

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

Copy link
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 pull request re-enables the GetAsync_UnicodeHostName_SuccessStatusCodeInResponse test that was disabled in 2019 due to an unreachable external Korean Starbucks website. The site (스타벅스코리아.com / xn--oy2b35ckwhba574atvuzkc.com) is now accessible again, allowing the test to exercise Unicode hostname to punycode IDN conversion. The test is re-enabled with an HTTP/3 exclusion since the site doesn't support that protocol.

Changes:

  • Removed the [ActiveIssue] attribute to re-enable the test
  • Added an HTTP/3 version check to skip the test when using HTTP/3, with explanatory comment

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.

GetAsync_UnicodeHostName_SuccessStatusCodeInResponse test failing in CI

3 participants

Comments