Closed mcdaddytalk closed 4 years ago
Hmm, I would propose that instead of putting this into the URL, you put the option of enabling SSL in config of Hikari. You should put it as a property of the datasource, instead of hardcoding it.
This looks good. Did you test it yourself on a MySQL server?
Yes. Tested on a Dev MC instance pointing to a local Mysql server. Tested different values as well for useSSL: true|false|blarg true failed because I do not have SSL set up but it did indicate a difference is settings. false passed blarg passed as it is not boolean and default "false" was used
Suggested fix for useSSL issue