Skip to content

Re-implement Kestrel's configuration loading using IOptions<T> #24091

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.

Sign up for GitHub

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

Jump to bottom
Open
davidfowl opened this issue Jul 18, 2020 · 2 comments
Open

Re-implement Kestrel's configuration loading using IOptions<T> #24091

davidfowl opened this issue Jul 18, 2020 · 2 comments
Labels
affected-very-few This issue impacts very few customers area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions not-on-roadmap severity-nice-to-have This label is used by an internal tool task
Milestone
Backlog

Comments

@davidfowl
Copy link
Member

Kestrel currently handles manual binding of configuration changes and instead should be using IOptionsMonitor<T> so that users can trigger changes to kestrel's configuration via other sources (rather than just binding to configuration changes).
@davidfowl
Copy link
Member Author

Something we should do in 6.

cc @halter73 @Tratcher

@BrennanConroy BrennanConroy added this to the Backlog milestone Jul 20, 2020
@ghost
Copy link

ghost commented Jul 20, 2020

We've moved this issue to the Backlog milestone. This means that it is not going to be worked on for the coming release. We will reassess the backlog following the current release and consider this item at that time. To learn more about our issue management process and to have better expectation regarding different types of issues you can read our Triage Process.

@halter73 halter73 added affected-very-few This issue impacts very few customers not-on-roadmap severity-nice-to-have This label is used by an internal tool task labels Nov 11, 2020 — with ASP.NET Core Issue Ranking
@davidfowl davidfowl mentioned this issue Dec 29, 2022
Closed
@amcasey amcasey added area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions and removed area-runtime labels Aug 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
affected-very-few This issue impacts very few customers area-networking Includes servers, yarp, json patch, bedrock, websockets, http client factory, and http abstractions not-on-roadmap severity-nice-to-have This label is used by an internal tool task
Projects
None yet
Milestone
Backlog
Development

No branches or pull requests
5 participants
@halter73 @davidfowl @BrennanConroy @amcasey @mkArtakMSFT