-
Notifications
You must be signed in to change notification settings - Fork 331
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
Remove TLS version fallback code #1409
Comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
In #458, code was added to try
TlsVersions.None
first, then fall back to TLS 1.1 if that failed. This was done to accommodate yaSSL-based servers (MySQL <= 5.7.28).This has caused multiple problems over the years:
The collection already contains item with same key 'net.transport'
exception #1405In addition, it makes debugging TLS connection issues harder:
Since this fix was only for a major version of MySQL Server that is no longer supported (5.7), there are minor versions that don't have the problem (>= 5.7.28), TLS 1.1 should generally be avoided, and there's now a
Tls Versions
connection string option that could be used as a workaround, the fallback code should simply be deleted and affected users advised to use the connection string setting.The text was updated successfully, but these errors were encountered: