aembke / redis-protocol.rs

A Rust implementation of RESP2 and RESP3
Apache License 2.0
39 stars 13 forks source link

fix: resp2 codec should be disabled if resp2 is disabled #32

Closed J0HN50N133 closed 5 months ago

J0HN50N133 commented 5 months ago

Currently, if you disabled resp2 and enable codec, resp2 will still be enabled, since codec implies resp2. However there is no reason to enable Resp2 if you already disable resp2.

aembke commented 5 months ago

Cherry-picking into https://github.com/aembke/redis-protocol.rs/pull/33