-
Notifications
You must be signed in to change notification settings - Fork 9.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Consume HostApplicationBuilder from WebApplicationBuilder #40459
Conversation
|
||
Properties = properties; | ||
foreach (var descriptor in _builder.Services) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I thought the goal was to remove the hacks...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I hope to propose a better public API than ConfigureWebHostDefaults to consume the default configuration from Microsoft.Extensions.Hosting in Microsoft.AspNetCore, but that will likely have a big surface area that will need API reviewed. I think this can be done as a follow up.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
3ce0067
to
680bf2a
Compare
5b1a369
to
ac89341
Compare
Depends on dotnet/runtime#65109 being merged and a subsequent arcade dependency update from the runtime repo.
I hope to propose a better public API than
ConfigureWebHostDefaults
(#40761) to consume the default configuration from Microsoft.Extensions.Hosting in Microsoft.AspNetCore, but that will likely have a big surface area that will need API reviewed. I think this can be done as a follow up.