Skip to content

Conversation

@perfectspr
Copy link

fix #295

@perfectspr perfectspr changed the title add 'validate' option to disable or enable hostname virification add 'validate' option to disable or enable hostname verification Mar 22, 2018
@timarmstrong
Copy link
Contributor

I think it would be clearer if the parameter name made it clear that it was verifying the SSL certificate. I looked at other dbapi drivers and everyone calls this parameter something different. ssl_verify_cert is MySql's choice and is unambiguous, so I think is a good choice - https://dev.mysql.com/doc/connector-python/en/connector-python-connectargs.html. Could you rename the "validate" argument to "ssl_verify_cert"?

@perfectspr
Copy link
Author

perfectspr commented Mar 22, 2019

renamed "validate" to "ssl_verify_cert"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

host name doesn't match when using LDAP and ca_cert

2 participants