Remove error caused by hiredis ssl not supported

[ke-miz]

SSL support in hiredis was merged three years ago. So no need for ssl error anymore.
#828

[stanhu]

@byroot @ke-miz This should be reverted. hiredis supports SSL, but hiredis-rb does not yet: redis/hiredis-rb#58 (comment)

[byroot]

You're right, but I disagree it should be reverted. If an adapter doesn't support something, it should raise itself, not rely on redis-rb to prevent that call from happening.

I'd rather leave it as is. And once I migrate redis-rb to use redis-client it won't matter anymore since it supports SSL with hiredis.

[stanhu]

Oh, great, redis-client looks much better than the current state of hiredis-rb.

If an adapter doesn't support something, it should raise itself, not rely on redis-rb to prevent that call from happening.

In this case, lib/redis/connection/hiredis.rb doesn't ever let hiredis-rb know that SSL is being used, so how would that work at the moment? I think hiredis-rb would need to be modified to take in the scheme or SSL options, and then raise the exception.

[byroot]

In this case, lib/redis/connection/hiredis.rb doesn't ever let hiredis-rb know that SSL is being used

Arf, right, I remembered the ssl: true being passed, but you're right it's not the case. I guess I'll revert ...