yii2-redis
yii2-redis copied to clipboard
Confusing "new" TLS scheme option vs old `useSSL`
2.0.18 added support for scheme tls in f96b872fa646308ab2f11721a7d35c713a234621
That adapts the connection string to tls:// instead of tcp://.
BUT we already had useSSL.
Imho there's a bunch of problems with having two options for kinda the same thing.
- what happens if I set scheme to
tlsanduseSSLto true? will thestream_socket_client()magically enable TLS but theuseSSLdo it again viastream_socket_enable_crypto()? Or will it throw an error? useSSL => trueshould probably also switch the scheme in theconnectionString, as that is a public getter/property- it's just plain confusing with two options