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

LoadConfigRoot doesn't handle config entries with null values #26

Closed
VitalyDukhota opened this issue Dec 14, 2018 · 1 comment
Closed
Assignees
Labels
bug Something isn't working P2

Comments

@VitalyDukhota
Copy link

VitalyDukhota commented Dec 14, 2018

There is a bug/issue in the .NET client for the new Ads API beta: ConfigBase.cs, LoadConfigRoot(IConfigurationRoot configurationRoot) method, line 57:

setting => setting.Value.ToString());

The problem is that setting.Value can for some config files be null, which is not handled correctly. Example - JSON configs with hierarchy, where a section name creates an entry in the config file, but it doesn't have a value.

@AnashOommen
Copy link
Member

Fixed in v0.7.1.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working P2
Projects
None yet
Development

No branches or pull requests

2 participants