Update to Synthetic User-Agent Telemetry Initializer to allow the con…

delradie commented 4 years ago

…figuration to optionally treat empty user agent strings as synthetic, as needed to handle FrontDoor health probes

Fix Issue #1297 As no solution to the blank useragent string has been forthcoming from the FrontDoor team I have updated the SyntheticUserAgentTelemetryInitializer to optionally treat empty user agents as synthetic

[ X] I ran Unit Tests locally.

For significant contributions please make sure you have completed the following items:

[ ] Changes in public surface reviewed
[ ] Design discussion issue #
[ ] CHANGELOG.md updated with one line description of the fix, and a link to the original issue.
[ ] The PR will trigger build, unit tests, and functional tests automatically. If your PR was submitted from fork - mention one of committers to initiate the build for you. If you want to to re-run the build/tests, the easiest way is to simply Close and Re-Open this same PR. (Just click 'close pull request' followed by 'open pull request' buttons at the bottom of the PR)
Please follow [these] (https://github.com/Microsoft/ApplicationInsights-dotnet-server/blob/develop/CONTRIBUTING.md) instructions to build and test locally.

msftclas commented 4 years ago

All CLA requirements met.

delradie commented 4 years ago

@delradie

microsoft / ApplicationInsights-dotnet-server

Update to Synthetic User-Agent Telemetry Initializer to allow the con… #1298