-
Notifications
You must be signed in to change notification settings - Fork 301
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
Add troubleshooting guidance for SQL Encrypt breaking change #5692
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Minor question: can we remove duplication from the guidance files?
Duplication (mostly) removed. I pulled the guidance out of the individual upgrade guides and created a specific upgrade guide for upgrading to SqlClient 4 in both SQL-T and SQL-P. The ServiceControl one has slightly different text so I left it as a "duplicate" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
minor nitpick
Microsoft.Data.SqlClient version 4 includes a breaking change dotnet/SqlClient#1210 which causes existing clients to throw an exception when using the default configuration: