hiredis-client 0.10.0 → 0.11.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8ab993c8bd2fc54e8817b047d657409ee3cf3ab8256884c052358cf4178d2835
-  data.tar.gz: 49298a0eedee98b454ece01d08de52d68291383ceccc5146f129cab4f2d87095
+  metadata.gz: 0f8f2c48819ebcd284e5de3249de6d76779f9d7914aa964eb78f630d846412ad
+  data.tar.gz: 9fa6140bf1d26b2f9a81df883c4242f73d519b274530379ec485f7babde9d99f
 SHA512:
-  metadata.gz: 1994b4d87d9b0ca699f4ab9a1b35d7f58834e1186ce8df4dbb57958b4a3a4e7db5379efa9e27c62ab01245c8d354babdd9a173234eaebfbb6e2187b8910ad419
-  data.tar.gz: 8bdfe9124ab8b8b491bd90b5effef54465bcf512c18fefe84c8af83dad396306d5cbd88e02ce84fd9db6835766bdbbcb055a2b3cc52a33cf149decbd802736f6
+  metadata.gz: 7a117bd47809ba3c8b1433607d240fd6dacf39fa253589b56506bee2435342afed569e45ef418880600cab67c231e1bc7a50cab62c135e9c1f834f824de9c0e0
+  data.tar.gz: 82d7ef4754a544649d1f999e196851ddea81e3a72ac418fc2a0bf3516301be5ef0b1b23cf9ab5bdb7d19958db483c4ba54dd973094800951e394ce0710868b4b

data/ext/redis_client/hiredis/export.clang

@@ -1,2 +1 @@
 _Init_hiredis_connection
-_ruby_abi_version

data/ext/redis_client/hiredis/extconf.rb

@@ -54,9 +54,13 @@ if RUBY_ENGINE == "ruby" && !RUBY_PLATFORM.match?(/mswin/)
     $CFLAGS << " -O3 "
   end
 
-  if `cc --version`.match?(/ clang /i) || RbConfig::CONFIG['CC'].match?(/clang/i)
+  cc_version = `#{RbConfig.expand("$(CC) --version".dup)}`
+  if cc_version.match?(/clang/i)
     $LDFLAGS << ' -Wl,-exported_symbols_list,"' << File.join(__dir__, 'export.clang') << '"'
-  elsif RbConfig::CONFIG['CC'].match?(/gcc/i)
+    if RUBY_VERSION >= "3.2"
+      $LDFLAGS << " -Wl,-exported_symbol,_ruby_abi_version"
+    end
+  elsif cc_version.match?(/gcc/i)
     $LDFLAGS << ' -Wl,--version-script="' << File.join(__dir__, 'export.gcc') << '"'
   end
 

data/ext/redis_client/hiredis/hiredis_connection.c

@@ -618,14 +618,17 @@ static VALUE hiredis_flush(VALUE self) {
     return Qtrue;
 }
 
+
+#define HIREDIS_FATAL_CONNECTION_ERROR -1
+#define HIREDIS_CLIENT_TIMEOUT -2
+
 static int hiredis_read_internal(hiredis_connection_t *connection, VALUE *reply) {
     void *redis_reply = NULL;
     int wdone = 0;
 
     /* Try to read pending replies */
     if (redisGetReplyFromReader(connection->context, &redis_reply) == REDIS_ERR) {
-        /* Protocol error */
-        return -1;
+        return HIREDIS_FATAL_CONNECTION_ERROR; // Protocol error
     }
 
     if (redis_reply == NULL) {
@@ -634,20 +637,19 @@ static int hiredis_read_internal(hiredis_connection_t *connection, VALUE *reply)
             errno = 0;
 
             if (hiredis_buffer_write_nogvl(connection->context, &wdone) == REDIS_ERR) {
-                /* Socket error */
-                return -1;
+                return HIREDIS_FATAL_CONNECTION_ERROR; // Socket error
             }
 
             if (errno == EAGAIN) {
                 int writable = 0;
 
                 if (hiredis_wait_writable(connection->context->fd, &connection->write_timeout, &writable) < 0) {
-                    return -2;
+                    return HIREDIS_CLIENT_TIMEOUT;
                 }
 
                 if (!writable) {
                     errno = EAGAIN;
-                    return -2;
+                    return HIREDIS_CLIENT_TIMEOUT;
                 }
             }
         }
@@ -657,20 +659,19 @@ static int hiredis_read_internal(hiredis_connection_t *connection, VALUE *reply)
             errno = 0;
 
             if (hiredis_buffer_read_nogvl(connection->context) == REDIS_ERR) {
-                /* Socket error */
-                return -1;
+                return HIREDIS_FATAL_CONNECTION_ERROR; // Socket error
             }
 
             if (errno == EAGAIN) {
                 int readable = 0;
 
                 if (hiredis_wait_readable(connection->context->fd, &connection->read_timeout, &readable) < 0) {
-                    return -2;
+                    return HIREDIS_CLIENT_TIMEOUT;
                 }
 
                 if (!readable) {
                     errno = EAGAIN;
-                    return -2;
+                    return HIREDIS_CLIENT_TIMEOUT;
                 }
 
                 /* Retry */
@@ -678,8 +679,7 @@ static int hiredis_read_internal(hiredis_connection_t *connection, VALUE *reply)
             }
 
             if (redisGetReplyFromReader(connection->context, &redis_reply) == REDIS_ERR) {
-                /* Protocol error */
-                return -1;
+                return HIREDIS_FATAL_CONNECTION_ERROR; // Protocol error
             }
         }
     }
@@ -697,9 +697,19 @@ static VALUE hiredis_read(VALUE self) {
     ENSURE_CONNECTED(connection);
 
     VALUE reply = Qnil;
-    if (hiredis_read_internal(connection, &reply)) {
-        hiredis_raise_error_and_disconnect(connection, rb_eRedisClientReadTimeoutError);
+    switch (hiredis_read_internal(connection, &reply)) {
+        case HIREDIS_FATAL_CONNECTION_ERROR:
+            // The error is unrecoverable, we eagerly close the connection to ensure
+            // it won't be re-used.
+            hiredis_raise_error_and_disconnect(connection, rb_eRedisClientReadTimeoutError);
+            break;
+        case HIREDIS_CLIENT_TIMEOUT:
+            // The timeout might have been expected (e.g. `PubSub#next_event`).
+            // we let the caller decide if the connection should be closed.
+            rb_raise(rb_eRedisClientReadTimeoutError, "Unknown Error");
+            break;
     }
+
     if (reply == Redis_Qfalse) {
         // See reply_create_bool
         reply = Qfalse;

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: hiredis-client
 version: !ruby/object:Gem::Version
-  version: 0.10.0
+  version: 0.11.1
 platform: ruby
 authors:
 - Jean Boussier
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-10-10 00:00:00.000000000 Z
+date: 2022-11-04 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.10.0
+        version: 0.11.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.10.0
+        version: 0.11.1
 description:
 email:
 - jean.boussier@gmail.com