Skip to content

Instantly share code, notes, and snippets.

@maryamariyan
Last active December 4, 2020 21:34
Show Gist options
  • Star 0 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save maryamariyan/a8606fbde9684f4e10d5631ca0d5a2a8 to your computer and use it in GitHub Desktop.
Save maryamariyan/a8606fbde9684f4e10d5631ca0d5a2a8 to your computer and use it in GitHub Desktop.
Triage notes on Microsoft.Extensions areas

Issues to consider for 6.0 on Extensions:

Presentation DI+HttpClientFactory

Meta:

  • Nullable annotations: #43605
  • API documentation debt on each of the areas: e.g #43871 and about 10 more similar.
  • General documentation regarding extensions: Currently docs are spread between both dotnet/docs and dotnet/AspNetCore.Docs. Would be good to drive an effort to ensure the ASP.NET Core docs have been sufficiently refactored to focus on web-specific examples and reduce duplication with docs in dotnet/docs (representing the basis for all .NET workloads, not just web), while ensuring the ASP.NET Core docs also maintain their own set of web focused content for Extensions.

Triage: Configuration

  • https://github.com/microsoft/reverse-proxy/blob/master/src/ReverseProxy/Utilities/ConfigurationReadingExtensions.cs bind... todo doc page

  • To be treated as maintainence mode. Features were complete for aspnet team. Unless we find improvements to make to support WPF/WinForms/Xamarin scenarios

  • Plan is to push back on new feature requests unless the can be opt-in behaviors. Generally it's pretty easy to implement your own configuration provider, so they always have a workaround with most open issues.

  • Found some issues dealing with the way we do key handling in configuration: (two escaping issues: #42643, #35989) and #40911

  • Investments that would make configuration better/faster/smarter add huge value here. (e.g. #43930, 36130, or perhaps #36010)

Triage Options:

Triage: DependencyInjection

  • Deadlock improvements, involves code refactoring
  • DI validation
  • DI diagnostics

Triage Hosting:

Trendy issues:

Triage: Logging

Driving work for VT100 API addition into System.Console:

  • Set foreground/background (TODO: create issue and add API proposal)

Driving work in System.Text.Json:

  • For making PooledByteBufferWriter public #33598

Triage FileSystem:

Driving work in area-System.IO:

Triage: Caching

We don't want to allow enumerating on keys while elemeents may get removed within another thread, (keeping as atomic operation). In order to allow for some of the feature requests in caching, (e.g. bulk remove) we can offer returning a snapshot (say array of elements. In this use case like wanting to remove with certain prefix "ZZZ", given the snapshot, will use TryGetValue and if already gone it would not throw. Below are the three related issues to consider:

  • How to flush all cache #36547
  • Bulk remove on IDistributedCache #36568 And also related to consider:
  • Enumerating keys using IDistributedCache #36402

Perf issue: (Future) dotnet/runtime#36499

Triage Primitives:

  • #36617 check proposed API that needs work
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment