Skip to content

Add HRESULT details to HttpSysListener initialization exception#66860

Open
MaikyOzr wants to merge 1 commit into
dotnet:mainfrom
MaikyOzr:main
Open

Add HRESULT details to HttpSysListener initialization exception#66860
MaikyOzr wants to merge 1 commit into
dotnet:mainfrom
MaikyOzr:main

Conversation

@MaikyOzr
Copy link
Copy Markdown

Add HRESULT details to HttpSysListener initialization exception

  • You've read the Contributor Guide and Code of Conduct.
  • You've included unit or integration tests for your change, where applicable.
  • You've included inline docs for your change, where applicable.
  • There's an open issue for the PR that you are making.

Description

When HttpInitialize fails, the resulting PlatformNotSupportedException
now includes the Win32 error code as an inner Win32Exception.
This makes it easier to diagnose why HTTP Server API initialization failed.

Fixes #66858

When HttpInitialize fails, include the Win32 error code as an inner
Win32Exception to make failure diagnosis easier.
@MaikyOzr MaikyOzr requested review from a team, BrennanConroy and halter73 as code owners May 26, 2026 23:14
@github-actions github-actions Bot added the area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions label May 26, 2026
@dotnet-policy-service dotnet-policy-service Bot added the community-contribution Indicates that the PR has been added by a community member label May 26, 2026
@dotnet-policy-service
Copy link
Copy Markdown
Contributor

Thanks for your PR, @MaikyOzr. Someone from the team will get assigned to your PR shortly and we'll get it reviewed.

@BrennanConroy
Copy link
Copy Markdown
Member

PR opened here #66859

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

Labels

area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions community-contribution Indicates that the PR has been added by a community member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add HttpInitialize HRESULT to HttpSysListener exception details

2 participants