search

microsoft / ApplicationInsights-dotnet-server

Microsoft Application Insights for .NET Web Applications
https://azure.microsoft.com/services/application-insights/
133 stars 67 forks source link

SQL Collection fix #1292

Closed cijothomas closed 4 years ago

cijothomas commented 4 years ago

Fixes #1291 SQL Listener had references to System.Data types in Microsoft.Data path.

The class is modified to NOT import the system.data namespace now to ensure we rely purely on reflection. This should have been done in the original fix which would have caught this issue there itself.

Fix Issue # .

- [ ] 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.