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

enabling to use double underscore delimiters #1

Merged
merged 2 commits into from
Apr 19, 2021
Merged

enabling to use double underscore delimiters #1

merged 2 commits into from
Apr 19, 2021

Conversation

mhirasawa-sansan
Copy link
Contributor

Thank you for developing a very useful and simple library.

In my project, I needed to use a hierarchical configuration, so I added it.
Using (__ ) for SecretID, covert to :. (It will be interpreted as a delimiter by the configuration provider https://github.com/aspnet/Configuration/blob/master/src/Config.Abstractions/ConfigurationPath.cs#L17)

This implementation is based on the following documentation and source code.
https://docs.microsoft.com/aspnet/core/fundamentals/configuration/?view=aspnetcore-5.0#environment-variables
https://github.com/aspnet/Configuration/blob/master/src/Config.EnvironmentVariables/EnvironmentVariablesConfigurationProvider.cs#L65-L68

Thank you.

@codecov-commenter
Copy link

Codecov Report

Merging #1 (438203c) into develop (6789ba8) will increase coverage by 0.52%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop       #1      +/-   ##
===========================================
+ Coverage    87.87%   88.40%   +0.52%     
===========================================
  Files            5        5              
  Lines           66       69       +3     
===========================================
+ Hits            58       61       +3     
  Misses           8        8              
Impacted Files Coverage Δ
...ger.Provider/SecretManagerConfigurationProvider.cs 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6789ba8...438203c. Read the comment docs.

@Cultti
Copy link
Owner

Cultti commented Apr 19, 2021

Great! Always glad to hear that someone is using this nuget :)

Could you also add test case for this?

Thank you!

@Cultti Cultti added the enhancement New feature or request label Apr 19, 2021
@mhirasawa-sansan
Copy link
Contributor Author

OK, I'll try it. please wait a moment.

@Cultti Cultti merged commit 9e72c97 into Cultti:develop Apr 19, 2021
Cultti added a commit that referenced this pull request Apr 19, 2021
Release 1.1.0.

Added features:
- Support multi level configurations with double underscores (__). Thank you mhirasawa-sansan! #1
@mhirasawa-sansan
Copy link
Contributor Author

I appreciate your kindness and support!
Thank you very much. 🤗

@mhirasawa-sansan mhirasawa-sansan deleted the feature/enable-delimiter branch April 20, 2021 02:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants