redis / lettucemod

Java client for Redis Modules
Apache License 2.0
49 stars 18 forks source link

Setting default protocol version to RESP2 when using spring #48

Closed zcw159357 closed 1 year ago

zcw159357 commented 1 year ago

the DEFAULT_CLIENT_OPTIONS in RedisModulesClient was overwritten in RedisModulesAutoConfiguration

zcw159357 commented 1 year ago

@jruaux Please review this PR. using spring boot with redis 7.2 RESP3 couldn't get response correctly from modules.

jruaux commented 1 year ago

Thanks for reporting this. I committed a fix that reuses the RedisModulesClient default options

zcw159357 commented 1 year ago

Thanks! And also there's another issue, spring boot 3.1.x changed RedisProperties and isSsl() no longer exists.