rage-iodine 3.3.0 → 4.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a1cf5cbcae299e1e28aaca04e99ea7ab9204a4d98fbf9434b27620f7588c22bc
-  data.tar.gz: 634a243fc8abdff8bfa59e21cf4e2a95a356e309569d17e85568cd5c2803d38f
+  metadata.gz: 26936e113c659cee25877d312515e6fe50ca5ed56ff5e8195e6f5274292f2145
+  data.tar.gz: 40b7aa9c791a5bc08513e5e0c478c02274a39c81d9be0fab7736f8478157ffab
 SHA512:
-  metadata.gz: e25730fe3ce0166e0e7ad4e260a1a1d15787cc264c906fce21a6a960614c3d5d033cbb597bb54cb36a97c316f8f64c5bd079e2c4d3f4cf5b6f0de1d890d92f27
-  data.tar.gz: fa94545855b466b9a16f00dbb267161b0cad66c546d973aa589bd74500868381c349e6d8df5d18c983fa3ecb4fd69fa5f21e57dbedc0f69c833c809c1eb4dc8f
+  metadata.gz: 1a789b92abed8d78b010ace73cb41d1688ff50c4fc6b590b6705a717196c212721039e252461ba8834a8f0bcd43679fc9c611a9f4d4690a826f3ac212bb3b9e5
+  data.tar.gz: 446a4ecf05a3b371266ea71a92cb514d9a510d2c0fb6dc72e212ff26253e60c343bfc1675a68e6f3c9985c267b9940e61492706f4e61d89e6cb73dbba6c05a2c

data/CHANGELOG.md

@@ -6,6 +6,12 @@ Please notice that this change log contains changes for upcoming releases as wel
 
 ## Changes:
 
+#### Change log v.4.0.0 (2024-09-11)
+
+**Update**: Stop disconnecting from the DB on fork.
+
+**Update**: Fiber Scheduler updates to correctly close file descriptors.
+
 #### Change log v.3.3.0 (2024-08-18)
 
 **Update**: Improvements and fixes for the static file service.

data/ext/iodine/fio.c

@@ -363,6 +363,8 @@ typedef struct {
   uint8_t open;
   /** indicated that the connection should be closed. */
   uint8_t close;
+  /** indicates whether the fd is coming from the Fiber Scheduler. */
+  uint8_t internal;
   /** peer address length */
   uint8_t addr_len;
   /** peer address */
@@ -2075,7 +2077,7 @@ static size_t fio_poll(void) {
         epoll_wait(internal[j].data.fd, events, FIO_POLL_MAX_EVENTS, 0);
     if (active_count > 0) {
       for (int i = 0; i < active_count; i++) {
-        if (events[i].events & (~(EPOLLIN | EPOLLOUT | EPOLLHUP | EPOLLRDHUP))) {
+        if (events[i].events & (~(EPOLLIN | EPOLLOUT | EPOLLHUP | EPOLLRDHUP | EPOLLERR))) {
           // errors are hendled as disconnections (on_close)
           fio_force_close_in_poll(fd2uuid(events[i].data.fd));
         } else {
@@ -2088,9 +2090,12 @@ static size_t fio_poll(void) {
             fio_defer_push_task(deferred_on_data,
                                 (void *)fd2uuid(events[i].data.fd), NULL);
           }
-          if (events[i].events & (EPOLLHUP | EPOLLRDHUP)) {
-            fio_defer_push_task(deferred_force_close_in_poll,
-                                (void *)fd2uuid(events[i].data.fd), NULL);
+          if (events[i].events & (EPOLLHUP | EPOLLRDHUP | EPOLLERR)) {
+            if(fd_data(events[i].data.fd).internal) {
+              fio_force_close_in_poll(fd2uuid(events[i].data.fd));
+            } else {
+              fio_clear_fd((intptr_t)events[i].data.fd, 0);
+            }
           }
         }
       } // end for loop
@@ -2226,8 +2231,11 @@ static size_t fio_poll(void) {
                             NULL);
       }
       if (events[i].flags & (EV_EOF | EV_ERROR)) {
-        fio_defer_push_task(deferred_force_close_in_poll,
-                            (void *)fd2uuid(events[i].udata), NULL);
+        if(fd_data(events[i].udata).internal) {
+          fio_force_close_in_poll(fd2uuid(events[i].udata));
+        } else {
+          fio_clear_fd((intptr_t)events[i].udata, 0);
+        }
       }
     }
   } else if (active_count < 0) {
@@ -4033,6 +4041,7 @@ static int fio_attach__internal(void *uuid_, void *protocol_) {
   }
   fio_protocol_s *old_pr = uuid_data(uuid).protocol;
   uuid_data(uuid).open = 1;
+  uuid_data(uuid).internal = 1;
   uuid_data(uuid).protocol = protocol;
   touchfd(fio_uuid2fd(uuid));
   fio_unlock(&uuid_data(uuid).protocol_lock);
@@ -4093,6 +4102,7 @@ static int fio_watch__internal(void *uuid_, void *protocol_) {
 
   fio_protocol_s *old_pr = uuid_data(uuid).protocol;
   uuid_data(uuid).open = 1;
+  uuid_data(uuid).internal = 0;
   uuid_data(uuid).protocol = protocol;
   touchfd(fio_uuid2fd(uuid));
   fio_unlock(&uuid_data(uuid).protocol_lock);

data/ext/iodine/scheduler.c

@@ -15,7 +15,8 @@
 static ID call_id;
 static uint8_t ATTACH_ON_READ_READY_CALLBACK;
 static uint8_t ATTACH_ON_WRITE_READY_CALLBACK;
-static VALUE timeout_args[1];
+static VALUE e_timeout_args[1];
+static VALUE e_closed_args[1];
 
 /* *****************************************************************************
 Fiber Scheduler API
@@ -36,6 +37,10 @@ typedef struct {
 static void iodine_scheduler_task_close(intptr_t uuid, fio_protocol_s *fio_protocol) {
   scheduler_protocol_s *protocol = (scheduler_protocol_s *)fio_protocol;
 
+  if (!protocol->fulfilled) {
+    IodineCaller.call2(protocol->block, call_id, 1, e_closed_args);
+  }
+
   IodineStore.remove(protocol->block);
   fio_free(protocol);
 
@@ -57,7 +62,7 @@ static void iodine_scheduler_task_timeout(intptr_t uuid, fio_protocol_s *fio_pro
   scheduler_protocol_s *protocol = (scheduler_protocol_s *)fio_protocol;
 
   if (!protocol->fulfilled) {
-    IodineCaller.call2(protocol->block, call_id, 1, timeout_args);
+    IodineCaller.call2(protocol->block, call_id, 1, e_timeout_args);
     protocol->fulfilled = 1;
   }
 }
@@ -178,7 +183,8 @@ Scheduler initialization
 
 void iodine_scheduler_initialize(void) {
   call_id = rb_intern2("call", 4);
-  timeout_args[0] = UINT2NUM(ETIMEDOUT);
+  e_timeout_args[0] = INT2NUM(-ETIMEDOUT);
+  e_closed_args[0] = INT2NUM(-EIO);
 
   VALUE SchedulerModule = rb_define_module_under(IodineModule, "Scheduler");
 

data/lib/iodine/version.rb

@@ -1,3 +1,3 @@
 module Iodine
-  VERSION = '3.3.0'.freeze
+  VERSION = '4.0.0'.freeze
 end

data/lib/iodine.rb

@@ -156,12 +156,6 @@ require 'rack/handler/iodine' unless defined? ::Iodine::Rack::IODINE_RACK_LOADED
 
 ### Automatic ActiveRecord and Sequel support for forking (preventing connection sharing)
 Iodine.on_state(:before_fork)  do
-  if defined?(ActiveRecord) && defined?(ActiveRecord::Base) && ActiveRecord::Base.respond_to?(:connection)
-    begin
-      ActiveRecord::Base.connection.disconnect!
-    rescue
-    end
-  end
   if defined?(Sequel)
     begin
       Sequel::DATABASES.each { |database| database.disconnect }
@@ -169,14 +163,6 @@ Iodine.on_state(:before_fork)  do
     end
   end
 end
-Iodine.on_state(:after_fork)  do
-  if defined?(ActiveRecord) && defined?(ActiveRecord::Base) && ActiveRecord::Base.respond_to?(:establish_connection)
-    begin
-      ActiveRecord::Base.establish_connection
-    rescue
-    end
-  end
-end
 
 ### Parse CLI for default HTTP settings
 Iodine::Base::CLI.parse if defined?(IODINE_PARSE_CLI) && IODINE_PARSE_CLI

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rage-iodine
 version: !ruby/object:Gem::Version
-  version: 3.3.0
+  version: 4.0.0
 platform: ruby
 authors:
 - Boaz Segev
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2024-08-18 00:00:00.000000000 Z
+date: 2024-09-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake