Improve Config Parsing Exceptions

[TimothyMothra]

Can we please change TelemetryConfigurationFactory to include the property name when it fails to parse a value from the configuration file.

Actual Exception:
System.FormatException: 'Input string was not in a correct format.'

Expected Exception:
Exception: Failed to parse 'MaxTransmissionBufferCapacity' Reason: 'Input string was not in a correct format'
Inner Exception: System.FormatException: 'Input string was not in a correct format.'

[cijothomas]

Where will user see this Exception?

[TimothyMothra]

it currently throws the FormatException in the user code when the config file is first parsed.

A sample app (see code sample) will throw on line 3 when it initializes a TelemetryClient.

        static void Main(string[] args)
        {
            var tc = new TelemetryClient();
            tc.TrackTrace("this is a test");
        }

[cijothomas]

i see. I never knew SDK would actually throw an exception but instead log to EventSource.
So now the proposed change throws a better exception, right/

[TimothyMothra]

Exactly. I'm just replacing an exception that is already thrown with more meaningful details.

[cijothomas]

Thanks for clarifying!