Fix string_or_nil_command to accept Null Array too

stefanwille / crystal-redis

Full featured Redis client for Crystal

MIT License

380 stars 61 forks source link

Fix string_or_nil_command to accept Null Array too #69

Closed maiha closed 6 years ago

maiha commented 6 years ago

Redis returns nil in two formats as written in https://redis.io/topics/protocol .

a) Null Bulk String ( "$-1\r\n" )
b) Null Array ( "*-1\r\n" )

In current master, string_or_nil_command accepts only the former pattern. And it will occur errors like #68 when the server returns nil in the latter pattern . This PR accepts both patterns.

Best regards,

stefanwille commented 6 years ago

Thanks!