IConfigurationSource change causes Providers Load() to be called twice ... by design?

[dtillman]

It looks to me that providers Load() method is being called twice now (RC2 bits). Once when the Build() method is called on ConfigurationBuilder and then again when ConfigurationRoot is constructed. Is this intended?

[HaoK]

Nope, we should pick one of these Load calls only. @divega have a preference which class owns calling Load?

[divega]

@dtillman good catch!

@HaoK it seems that ConfigurationRoot already has logic to load providers because of Reload(), and it seems cleaner to constrain the responsibility of the builder to just build the list of providers and pass it to the root, so I would vote for removing the call to Load() in the builder.

[HaoK]

9a8825a