Skip to content
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

Configuration sync disposable config providers #31

Merged

Conversation

jf-06
Copy link
Contributor

@jf-06 jf-06 commented Oct 4, 2024

No description provided.

jf-06 added 3 commits October 3, 2024 18:13
All packages are now on inline versioning.
~ Convert LogLevel property into a getter for dynamic runtime fetching of the log level.
~ Add a fix on Windows for color logging failing by manually updating the console mode flags with ENABLE_VIRTUAL_TERMINAL_PROCESSING.
~ Rename LogLevel.Trace and ILogger.Trace to Verbose and add better docomments to each method and log level to better explain their use cases.
~ Remove ILogger.Log as this was technically an alias to ILogger.Information as it did the same thing.
~ Update all log methods to now take the raw message getter instead of formatting the messages themselves, this will help to introduce better performance by not evaluating the message straight away.
~ Add another ILogger.Error method with an argument to supply a message that precedes the exception trace back.
Convert IVaultProvider to IDisposable
Update current registered auto refresh providers to a dictionary to make concurrent removal easier.
Expose the current cached values to inheritied classes.
Convert mount and path to virutal to not introduce a breaking change by adding set to the mix.
Move the refresh thread from a simple thread wait to a wait event system to allow manual refreshing of all global providers.
Add argument to constructors to control if the provider should be added to the auto refresh thread or not.
Add extra constructor to allow setting mount and path via constructor.
Implement IDisposable
@jf-06 jf-06 merged commit eeca03d into master Oct 4, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant