Add option to remove location data from telemetry events

[aeisenberg]

For our extension, we want to gather as little information as we can about our users while still being able to answer the questions we have.

With this in mind, we do not need any location data. I would like to add an option to remove this from the payload. I can see several possible solutions:

1. add a new flag to disable collecting location data (easy for users, but not very flexible if users also want to void sending other data, like client IP).
2. expose the internal applicationInsight instance and allow users to customize in any way they want.
3. Allow users to add custom TelemetryProcessors to the applicationInsight instance.

Any of these solutions would be fine by me.

[sbatten]

good thought! isn't client ip already being dropped by appinsights?

[aeisenberg]

Client IP was actually a bad example on my part. It looks like the applicationInsight has no control over collecting the IP as far as I can tell. I'm guessing the client IP is somehow extracted from the header of the HTTP request to azure. Though, I'm not exactly sure.