-
Notifications
You must be signed in to change notification settings - Fork 10.7k
Use runtime property settings instead of environment variables for setting startup hook #9520
Copy link
Copy link
Closed
Labels
acceptedThis issue has completed "acceptance" testing (including accessibility)This issue has completed "acceptance" testing (including accessibility)area-networkingIncludes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractionsIncludes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractionsenhancementThis issue represents an ask for new feature or an enhancement to an existing oneThis issue represents an ask for new feature or an enhancement to an existing onefeature-iisIncludes: IIS, ANCMIncludes: IIS, ANCM
Milestone
Metadata
Metadata
Assignees
Labels
acceptedThis issue has completed "acceptance" testing (including accessibility)This issue has completed "acceptance" testing (including accessibility)area-networkingIncludes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractionsIncludes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractionsenhancementThis issue represents an ask for new feature or an enhancement to an existing oneThis issue represents an ask for new feature or an enhancement to an existing onefeature-iisIncludes: IIS, ANCMIncludes: IIS, ANCM
Type
Fields
Give feedbackNo fields configured for issues without a type.
Epic #8833
The startup hook today is set via environment variable right now (https://github.com/aspnet/AspNetCore/blob/master/src/Servers/IIS/AspNetCoreModuleV2/InProcessRequestHandler/inprocessapplication.cpp#L241). Core-setup will eventually be adding an api to set runtime properties like https://github.com/dotnet/core-setup/pull/5336/files#diff-15fec1485f5f10c590d5b840300283ceR233. We should use these instead.