New Ads API Beta - .NET client bug

[Vitaly Dukhota]

Hi, I am not sure if it’s the best place to report a problem – please point me in the right direction if it isn’t.

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.

Also a side note: current GoogleAdsConfig implementation is tied to ConfigurationManager, which is only used by .NET Framework for desktop programs. .NET Web applications, and specifically .NET Core applications use WebConfigurationManager which the current implementation of GoogleAdsConfig ignores, and so it doesn't pick up configuration from it. It's not a big deal because there is LoadConfigRoot(IConfigurationRoot configurationRoot) mentioned above, which allows to load configuration from anywhere - but it requires a bit of custom coding. Just thought I would let you know.

[Teja Makani]

Hello Vitaly,

As this issue is specific to client libraries, could you please create a new issue on the .Net client library from here. Client library owners are better equipped to address this issue.

Regards,
Sai Teja, AdWords API Team