Bug: Both TelemetryChannels do not correctly handle AggregateException

If you are reporting bug/issue, please provide detailed Repro instructions.

## Repro Steps
**still investigating, don't have a repro yet**

Breeze fixed their issues. Can test using this: https://badssl.com/

##  Actual Behavior

We aren't handling the `AggregateException`.
Instead, we log this message: 
> "Failed to send: One or more errors occurred.."


https://github.com/microsoft/ApplicationInsights-dotnet/blob/f181abbcdf6a6a2a4aea821ce1d72e98b0c68fbe/src/Microsoft.ApplicationInsights/Channel/InMemoryTransmitter.cs#L136-L144

Send method returns a Task
https://github.com/microsoft/ApplicationInsights-dotnet/blob/f181abbcdf6a6a2a4aea821ce1d72e98b0c68fbe/src/Microsoft.ApplicationInsights/Channel/InMemoryTransmitter.cs#L151



## Expected Behavior

We need to correctly handle the `AggregateException`.



## Version Info 
SDK Version : all
.NET Version :	all						
How Application was onboarded with SDK(VisualStudio/StatusMonitor/Azure Extension) :
OS : nuget
Hosting Info (IIS/Azure WebApps/ etc) : onprem

	try
	{
	// send request
	this.Send(telemetryItems, timeout).Wait();
	}
	catch (Exception e)
	{
	CoreEventSource.Log.FailedToSend(e.Message);
	}

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Bug: Both TelemetryChannels do not correctly handle AggregateException #1173

TimothyMothra
openedon Jul 19, 2019

Repro Steps

Actual Behavior

Expected Behavior

Version Info

Assignees

Labels

Type

Projects

Milestone

Relationships

Development