Closed rockfordlhotka closed 5 months ago
This isn't complete. It is my first attempt at using the new GH Copilot workspace preview feature. Not entirely sure the AI did the right thing, and in any case, it didn't generate (or allow me to add) any unit tests.
Fixes #4047
This pull request addresses the issue of
ApplicationContextAccessor
not selecting theContextManager
specified inCslaOptions.ContextManagerType
over the default one. The changes ensure that customContextManager
types are prioritized and registered before the default types, allowing for the correctContextManager
to be used according to the configuration.RegisterContextManager
method inConfigurationExtensions.cs
to useTryAddScoped
instead ofAddScoped
for registering defaultContextManager
types. This change prevents overwriting custom registrations if a customContextManagerType
is provided inCslaOptions
.ContextManagerType
is specified, it is registered before attempting to register any defaultContextManager
types, allowing the custom type to take precedence.TryAddScoped
in theLoadContextManager
method for loading specificContextManager
types based on the environment, further ensuring that custom types are not overwritten by default or environment-specific types.For more details, open the Copilot Workspace session.