bart-vmware commented 2 months ago

Placeholder/encryption configuration providers

Extract duplicate code from configuration sources/providers
Fix reloading and change notification when binding to options
Remove special-cased code paths for ConfigurationManager (no longer needed)
Add trace-level logging, using source generator
Refactor internal methods to recursively search through configuration sources/providers and update call sites
Wire Configuration.Encryption from AddSteeltoe in Bootstrap

Placeholder

Fixes in placeholder substitution when using '.' (Spring-style) key separator with default value

Rename various types and methods containing Encrypt* that actually decrypt.
Remove IConfiguration extension methods, lazily create decryptor from loaded configuration
Fix regex used in decryption, tweak its options, and use source generator

Change: Do not implicitly add placeholder anymore; nesting of placeholder/encryption depends on what the user needs
Remove duplicate tests

Fix host configuration from not being included in the response
Include an empty placeholder/encryption property source in the response when used

Fixes #830.

[x] Your code complies with our Coding Style.
[x] You've updated unit and/or integration tests for your change, where applicable.
[x] You've updated documentation for your change, where applicable. If your change affects other repositories, such as Documentation, Samples and/or MainSite, add linked PRs here.
[x] There's an open issue for the PR that you are making. If you'd like to propose a new feature or change, please open an issue to discuss the change or find an existing issue.
[x] You've added required license files and/or file headers (explaining where the code came from with proper attribution), where code is copied from StackOverflow, a blog, or OSS.