New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Regression | Revert PR #2281 SNIProxy code change #2395
Conversation
src/Microsoft.Data.SqlClient/netcore/src/Microsoft/Data/SqlClient/SNI/SNIProxy.cs
Outdated
Show resolved
Hide resolved
src/Microsoft.Data.SqlClient/netcore/src/Microsoft/Data/SqlClient/SNI/SNIProxy.cs
Outdated
Show resolved
Hide resolved
Co-authored-by: David Engel <dengel1012@gmail.com>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2395 +/- ##
==========================================
- Coverage 72.69% 72.62% -0.07%
==========================================
Files 310 310
Lines 61885 61885
==========================================
- Hits 44987 44947 -40
- Misses 16898 16938 +40
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Can you link the related issues for the record? |
Wonder if it would be possible to add a test to prevent this from regressing again? |
I was thinking the same. I compared it with SDS and will start by adding some tests. However, I need to understand the reason behind #2394 first. It seems the underlying cause of that issue might be a bit different. |
This PR reverts back the code change from #2281 to address regression in v5.2.0