Do not prevent correlation headers injection on localhost (#1122)

CHANGELOG.md

@@ -1,6 +1,11 @@
 # Changelog
 
 ## Version 2.9.0-beta3
+- [Fix: Correlation doesn't work for localhost](https://github.com/Microsoft/ApplicationInsights-dotnet-server/issues/1120). If you are upgrading and have previously opted into legacy header injection via `DependencyTrackingTelemetryModule.EnableLegacyCorrelationHeadersInjection` and run app locally with Azure Storage Emulator, make sure you manually exclude localhost from correlation headers injection in the `ExcludeComponentCorrelationHttpHeadersOnDomains` under `DependencyCollector`
+    ```xml
+        <Add>localhost</Add>
+        <Add>127.0.0.1</Add>
+    ```
 - [Fix: Non-default port is not included into the target for Http dependencies on .NET Core](https://github.com/Microsoft/ApplicationInsights-dotnet-server/issues/1121)
 - [When Activity has root id compatible with W3C trace Id, use it as trace id](https://github.com/Microsoft/ApplicationInsights-dotnet-server/pull/1107)
 

Src/DependencyCollector/DependencyCollector/ApplicationInsights.config.install.xdt

@@ -14,8 +14,6 @@
         <Add>core.chinacloudapi.cn</Add>
         <Add>core.cloudapi.de</Add>
         <Add>core.usgovcloudapi.net</Add>
-        <Add>localhost</Add>
-        <Add>127.0.0.1</Add>
       </ExcludeComponentCorrelationHttpHeadersOnDomains>
       <IncludeDiagnosticSourceActivities>
         <Add>Microsoft.Azure.EventHubs</Add>