Application Insights config from appsettings.json

[daveta]

John, here's a stab at "no botfile" override.

Would logically change:
public void ConfigureServices(IServiceCollection services)
{
var botConfig = BotConfiguration.Load("testbot.bot", null);
services.AddBotApplicationInsights(botConfig);

To something like:
public IConfiguration Configuration { get; }

public void ConfigureServices(IServiceCollection services)
{
services.AddBotApplicationInsights(Configuration);

[coveralls]

Pull Request Test Coverage Report for Build 50028

• 0 of 0 changed or added relevant lines in 0 files are covered.
• 6 unchanged lines in 1 file lost coverage.
• Overall coverage increased (+0.05%) to 74.605%

---

Files with Coverage Reduction	New Missed Lines	%
/libraries/integration/Microsoft.Bot.Builder.Integration.ApplicationInsights.Core/ServiceCollectionExtensions.cs	6	86.15%

Totals
Change from base Build 49945:	0.05%
Covered Lines:	4010
Relevant Lines:	5375

---

💛 - Coveralls

[cleemullins]

@daveta The approach @johnataylor is using in the samples is quite clean and seems like it would directly apply here. Let's sync tomorrow and see if we can get this closed across the board.

... and we need to make sure we don't break compat as we do it.

[cleemullins]

This LGTM, but let's sync with @johnataylor before we do anything here.

[cleemullins]

Added "Blocked" just so this isn't accidentally merged until John can confirm this matches what he's doing...

[daveta]

This is actually just using what the Application Insights default appsettings.json keys are. If we want to deviate from that, we can align on whatever is decided..

[daveta]

FYI: I believe @darrenj was going to take for his work (with the "standard" App Insights appsettings.json).

One thing I haven't investigated with the "standard" settings is how to support multiple distinct Application Insights instances. Something we could support in the ..bot config file.