sewenew
sewenew
Hi @mfs-git YES, the current implementation of `parse` doesn't modify `redisReply` object, and it can be `const`. Since when we parse `redisReply` to `std::string`, we copy the underlying dynamically allocated...
Hi @mfs-git > Actually this unnecessary non-const-ness will propagate from this library into application code Can you show me some code example, why this `parse` function will be propagate to...
@Cezero Thanks for your suggestion! When connection is in subscribing mode, it's restricted, and can only call a few commands. So `Subscriber::set_name()` might be hard to be used correctly, since...
@Cezero Sorry for the late reply! The first version of this feature has been implemented, and you can check out the *setname* branch and take a try: ``` ConnectionOptions opts;...
Since you enabled `ConnectionOptions::tls::enable`, you need to specify other TLS options, e.g. certification file, key file. Please check the doc for detail. > I tried to connect to the same...
@adobeturchenko Thanks for pointing it out! It seems that this is a new feature for redis-cli, and it has not been supported by [hiredis](https://github.com/redis/hiredis/blob/d6a0b192b414dc261589d4c2d16577c2192b39c0/ssl.c#L238): ``` SSL_CTX_set_verify(ctx->ssl_ctx, SSL_VERIFY_PEER, NULL); ``` So...
I think it's better to make hiredis to support it, so that redis-plus-plus can be consistent with it. I've created a pull request to hiredis: [#927](https://github.com/redis/hiredis/issues/927). When it's done, I'll...
@wingunder I was plan to add Docker support, including Redis, Redis Cluster, Redis Sentinel, and related test code for them. However, I'm too busy to make it work. I'm so...
Hi @langep Sorry for the late reply. These days I'm busy working on a new open source project. I'm not quite sure if these changes have some side effect, e.g....
Hi @langep I tried the `git submodule` commands. It's great, and makes it easier to manage the dependency. However, when I run `make install`, it will also overwrite the pre-installed...