hiredis-client 0.14.1 → 0.16.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3506adf6e5b2e5da54f4fc75e762a0fd4f23b9bd9ea225a468b2ae96360d0273
-  data.tar.gz: 484b29d8fb080fad9076e71f1db9a1703b9b2a3a5ae173b6a5073d5abb2bfbe6
+  metadata.gz: 61bc01f80337772a560ed39e2ee0ecf4c783f9190dba9d58d8a4a4b9508b2ec6
+  data.tar.gz: 9ea97d9bf65cf629ca7501534faa2644dd8c8afc2e9d74b973d8d661909cd327
 SHA512:
-  metadata.gz: 3e238fe1e94a808f6c1966c8c2ce1136fa173594cfa824156199c3509b3b61c8b9f97a75336458edf7c50dcddc7a7d08833358a9f34bf64f141587fb1f19fa74
-  data.tar.gz: a6ddda3ab9d41f591cd82c1319fd43fc701cbb0c515768e0d940bc095fc69c69334bbf932c860ab37944a7f73e141a5e6b25fc4a2de454fb726f4dd199fadf48
+  metadata.gz: e334396d8518211eda9f1c60151b140cc713049ec32b68b57de0f16977c76693b3a4de7b93c70a56dbb1ef1826717754d0e8ccc92247c7f26f01e853f8093ea0
+  data.tar.gz: 9e3597f5cde32b9289c5d35554046b4e40bda564656eb94dafe65521e22049baa19decbda89b215343187e054a30372ed7d2c96f9297dbfb285ac9ab10e1fc2b

data/ext/redis_client/hiredis/hiredis_connection.c

@@ -39,6 +39,7 @@
 #include "hiredis.h"
 #include "net.h"
 #include "hiredis_ssl.h"
+#include <poll.h>
 
 #if !defined(RUBY_ASSERT)
 #  define RUBY_ASSERT(condition) ((void)0)
@@ -797,6 +798,93 @@ static VALUE hiredis_close(VALUE self) {
     return Qnil;
 }
 
+static inline double diff_timespec_ms(const struct timespec *time1, const struct timespec *time0) {
+  return ((time1->tv_sec - time0->tv_sec) * 1000.0)
+      + (time1->tv_nsec - time0->tv_nsec) / 1000000.0;
+}
+
+static inline int timeval_to_msec(struct timeval duration) {
+    return duration.tv_sec * 1000 + duration.tv_usec / 1000;
+}
+
+typedef struct {
+    hiredis_connection_t *connection;
+    struct timespec start;
+    struct timespec end;
+    int return_value;
+} hiredis_measure_round_trip_delay_args_t;
+
+static const size_t pong_length = 7;
+
+static void *hiredis_measure_round_trip_delay_safe(void *_args) {
+    hiredis_measure_round_trip_delay_args_t *args = _args;
+    hiredis_connection_t *connection = args->connection;
+    redisReader *reader = connection->context->reader;
+
+    if (reader->len - reader->pos != 0) {
+        args->return_value = REDIS_ERR;
+        return NULL;
+    }
+
+    redisAppendFormattedCommand(connection->context, "PING\r\n", 6);
+
+    clock_gettime(CLOCK_MONOTONIC, &args->start);
+
+    int wdone = 0;
+    do {
+        if (redisBufferWrite(connection->context, &wdone) == REDIS_ERR) {
+            args->return_value = REDIS_ERR;
+            return NULL;
+        }
+    } while (!wdone);
+
+    struct pollfd   wfd[1];
+    wfd[0].fd     = connection->context->fd;
+    wfd[0].events = POLLIN;
+    int retval = poll(wfd, 1, timeval_to_msec(connection->read_timeout));
+    if (retval == -1) {
+        args->return_value = REDIS_ERR_IO;
+        return NULL;
+    } else if (!retval) {
+        args->return_value = REDIS_ERR_IO;
+        return NULL;
+    }
+
+    redisBufferRead(connection->context);
+
+    if (reader->len - reader->pos != pong_length) {
+        args->return_value = REDIS_ERR;
+        return NULL;
+    }
+
+    if (strncmp(reader->buf + reader->pos, "+PONG\r\n", pong_length) != 0) {
+        args->return_value = REDIS_ERR;
+        return NULL;
+    }
+    reader->pos += pong_length;
+
+    clock_gettime(CLOCK_MONOTONIC, &args->end);
+    args->return_value = REDIS_OK;
+    return NULL;
+}
+
+static VALUE hiredis_measure_round_trip_delay(VALUE self) {
+    CONNECTION(self, connection);
+    ENSURE_CONNECTED(connection);
+
+    hiredis_measure_round_trip_delay_args_t args = {
+        .connection = connection,
+    };
+    rb_thread_call_without_gvl(hiredis_measure_round_trip_delay_safe, &args, RUBY_UBF_IO, 0);
+
+    if (args.return_value != REDIS_OK) {
+        hiredis_raise_error_and_disconnect(connection, rb_eRedisClientReadTimeoutError);
+        return Qnil; // unreachable;
+    }
+
+    return DBL2NUM(diff_timespec_ms(&args.end, &args.start));
+}
+
 RUBY_FUNC_EXPORTED void Init_hiredis_connection(void) {
     // Qfalse == NULL, so we can't return Qfalse in `reply_create_bool()`
     RUBY_ASSERT((void *)Qfalse == NULL);
@@ -843,6 +931,7 @@ RUBY_FUNC_EXPORTED void Init_hiredis_connection(void) {
     rb_define_private_method(rb_cHiredisConnection, "_read", hiredis_read, 0);
     rb_define_private_method(rb_cHiredisConnection, "flush", hiredis_flush, 0);
     rb_define_private_method(rb_cHiredisConnection, "_close", hiredis_close, 0);
+    rb_define_method(rb_cHiredisConnection, "measure_round_trip_delay", hiredis_measure_round_trip_delay, 0);
 
     VALUE rb_cHiredisSSLContext = rb_define_class_under(rb_cHiredisConnection, "SSLContext", rb_cObject);
     rb_define_alloc_func(rb_cHiredisSSLContext, hiredis_ssl_context_alloc);

data/lib/redis_client/hiredis_connection.rb

@@ -10,6 +10,17 @@ class RedisClient
 
     class << self
       def ssl_context(ssl_params)
+        unless ssl_params[:ca_file] || ssl_params[:ca_path]
+          default_ca_file = OpenSSL::X509::DEFAULT_CERT_FILE
+          default_ca_path = OpenSSL::X509::DEFAULT_CERT_DIR
+
+          if File.readable? default_ca_file
+            ssl_params[:ca_file] = default_ca_file
+          elsif File.directory? default_ca_path
+            ssl_params[:ca_path] = default_ca_path
+          end
+        end
+
         HiredisConnection::SSLContext.new(
           ca_file: ssl_params[:ca_file],
           ca_path: ssl_params[:ca_path],

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: hiredis-client
 version: !ruby/object:Gem::Version
-  version: 0.14.1
+  version: 0.16.0
 platform: ruby
 authors:
 - Jean Boussier
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-03-30 00:00:00.000000000 Z
+date: 2023-08-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: redis-client
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.14.1
+        version: 0.16.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.14.1
+        version: 0.16.0
 description:
 email:
 - jean.boussier@gmail.com
@@ -105,7 +105,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.4.6
+rubygems_version: 3.4.10
 signing_key:
 specification_version: 4
 summary: Hiredis binding for redis-client