Skip to content

Set a default for the Http3Support feature switch#117081

Closed
MichalStrehovsky wants to merge 1 commit intomainfrom
MichalStrehovsky-patch-1
Closed

Set a default for the Http3Support feature switch#117081
MichalStrehovsky wants to merge 1 commit intomainfrom
MichalStrehovsky-patch-1

Conversation

@MichalStrehovsky
Copy link
Member

Control over this is moving from dotnet/runtime repo build time to publish time so we need to set a default that matches the old build-time setting.

Ref #117012
Ref dotnet/sdk#49564

Control over this is moving from dotnet/runtime repo build time to publish time so we need to set a default that matches the old build-time setting.

Ref #117012
Ref dotnet/sdk#49564
Copilot AI review requested due to automatic review settings June 27, 2025 09:52
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 PR sets a default value for the Http3Support feature switch to false in the WasmFeatures.props file, aligning the publish time behavior with the previous build-time setting.

  • Default for Http3Support added.
  • Consistency with previous dotnet/runtime behavior ensured.

@github-actions github-actions bot added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Jun 27, 2025
@akoeplinger
Copy link
Member

This file moved to https://github.com/dotnet/sdk/blob/main/src/Workloads/Manifests/Microsoft.NET.Workload.Mono.Toolchain.Current.Manifest/WasmFeatures.props. Looks like I missed removing it here, I'll get that done.

@akoeplinger akoeplinger deleted the MichalStrehovsky-patch-1 branch June 27, 2025 10:39
@MichalStrehovsky
Copy link
Member Author

This file moved to https://github.com/dotnet/sdk/blob/main/src/Workloads/Manifests/Microsoft.NET.Workload.Mono.Toolchain.Current.Manifest/WasmFeatures.props. Looks like I missed removing it here, I'll get that done.

Ah, thanks. New PR at dotnet/sdk#49583.

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

Labels

needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants