itsi 0.2.18 → 0.2.19

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d643690ccc0157ece254b12b72bd4eff904a907db970639df8567c43693b2c7b
-  data.tar.gz: 5202c977ca30f83ddc09fc9bbe31966b06ffb5beda308568a684e0242d3898f7
+  metadata.gz: 8707abd045843bf5567c62f0f458db47d582d5e711916627d93b7c8b68bec8f6
+  data.tar.gz: a0d55d6f5048ffc3f5d779ab3fe46ffa89f6994af18fbaf1dad908f51b80ff74
 SHA512:
-  metadata.gz: 07567a207915ae2f6d4dc0c6827e5b9008574f1e8b9864b0969657a06d5f96f4d638b0532187da82511b7f71b06c7bc1499fd1cd32cb7e85fa13c3cb0d76e34a
-  data.tar.gz: 0f7bfcc95d1a79deae127f6899122c8ffd3d3042332ad33bbc2cf0e490abb9392dfbce76f4cbcfa878b58ef8e6ee8b3694853994c9c78dbc2b4165fe76bf6be5
+  metadata.gz: d649294f7eae29ea5dee877f52a6f6e255cd3b4c7eca008e8ff8670d996b206ae9dedf04fdba3c80540b9fbee1b072df2f60e03d363b4868b2cac8e69679a798
+  data.tar.gz: 233febdc6634ee20ff69228dcf8a0c7c42633461d98da67cb7391e2d3beef5e736e8843034a56cbe897d8a9f51993371c9d8c234cb3e9fddd27f7f11455b0b69

data/CHANGELOG.md

@@ -1,10 +1,14 @@
-## [0.2.18] - 2025-XX-XX
-### WIP
--- Fixing error in auto-reload on Linux when reuse_port is false
--- Fix breaking of auto-reload on config file errors
--- include directive is relative (equivalent to require_relative)
--- Fixing preload gem group logic
--- Fix errors in interrupt handling during some debug flows
+## [0.2.19] - 2025-08-09
+- Fixed shutdown hook incorrectly being skipped
+- Documentation fixes
+- Fixed stuck graceful shutdown scenario
+
+## [0.2.18] - 2025-06-20
+- Fixing error in auto-reload on Linux when reuse_port is false
+- Fix breaking of auto-reload on config file errors
+- include directive is relative (equivalent to require_relative)
+- Fixing preload gem group logic
+- Fix errors in interrupt handling during some debug flows
 
 ## [0.2.17] - 2025-05-31
 - Enabled vectorized writes in IoSteam

data/Cargo.lock

@@ -1644,7 +1644,7 @@ checksum = "4a5f13b858c8d314ee3e8f639011f7ccefe71f97f96e50151fb991f267928e2c"
 
 [[package]]
 name = "itsi-scheduler"
-version = "0.2.18"
+version = "0.2.19"
 dependencies = [
  "bytes",
  "derive_more",
@@ -1662,7 +1662,7 @@ dependencies = [
 
 [[package]]
 name = "itsi-server"
-version = "0.2.18"
+version = "0.2.19"
 dependencies = [
  "argon2",
  "async-channel",

data/crates/itsi_scheduler/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "itsi-scheduler"
-version = "0.2.18"
+version = "0.2.19"
 edition = "2021"
 authors = ["Wouter Coppieters <wc@pico.net.nz>"]
 license = "MIT"

data/crates/itsi_scheduler/extconf.rb

@@ -1,8 +1,11 @@
 # frozen_string_literal: true
 
-require "mkmf"
-require "rb_sys/mkmf"
+require 'mkmf'
+require 'rb_sys/mkmf'
 
-create_rust_makefile("itsi/scheduler/itsi_scheduler") do |r|
-  r.extra_rustflags = ["-C target-cpu=native"]
+create_rust_makefile('itsi/scheduler/itsi_scheduler') do |r|
+  r.extra_rustflags = ['-C target-cpu=native']
+  r.env = {
+    'BINDGEN_EXTRA_CLANG_ARGS' => '-include stdbool.h -std=c99'
+  }
 end

data/crates/itsi_server/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "itsi-server"
-version = "0.2.18"
+version = "0.2.19"
 edition = "2021"
 authors = ["Wouter Coppieters <wc@pico.net.nz>"]
 license = "MIT"

data/crates/itsi_server/extconf.rb

@@ -1,8 +1,11 @@
 # frozen_string_literal: true
 
-require "mkmf"
-require "rb_sys/mkmf"
+require 'mkmf'
+require 'rb_sys/mkmf'
 
-create_rust_makefile("itsi/server/itsi_server") do |r|
-  r.extra_rustflags = ["-C target-cpu=native"]
+create_rust_makefile('itsi/server/itsi_server') do |r|
+  r.extra_rustflags = ['-C target-cpu=native']
+  r.env = {
+    'BINDGEN_EXTRA_CLANG_ARGS' => '-include stdbool.h -std=c99'
+  }
 end

data/crates/itsi_server/src/server/serve_strategy/cluster_mode.rs

@@ -377,9 +377,13 @@ impl ClusterMode {
               }
               lifecycle_event = lifecycle_rx.recv() => match lifecycle_event{
                 Ok(lifecycle_event) => {
+                  debug!("Cluster mode received lifecycle event: {:?}", lifecycle_event);
                   if let Err(e) = self_ref.clone().handle_lifecycle_event(lifecycle_event).await{
                     match e {
-                      ItsiError::Break => break,
+                      ItsiError::Break => {
+                        debug!("Lifecycle event triggered shutdown, breaking cluster monitor loop");
+                        break;
+                      },
                       _ => error!("Error in handle_lifecycle_event {:?}", e)
                     }
                   }

data/crates/itsi_server/src/server/signal.rs

@@ -1,11 +1,12 @@
 use std::{
     collections::VecDeque,
-    sync::atomic::{AtomicBool, AtomicI8},
+    sync::atomic::{AtomicBool, AtomicI8, Ordering},
 };
 
 use nix::libc::{self, sighandler_t};
 use parking_lot::Mutex;
 use tokio::sync::broadcast;
+use tracing::{debug, warn};
 
 use super::lifecycle_event::LifecycleEvent;
 
@@ -21,12 +22,14 @@ pub fn subscribe_runtime_to_signals() -> broadcast::Receiver<LifecycleEvent> {
     if let Some(sender) = guard.as_ref() {
         return sender.subscribe();
     }
-    let (sender, receiver) = broadcast::channel(5);
+    let (sender, receiver) = broadcast::channel(32);
     let sender_clone = sender.clone();
     std::thread::spawn(move || {
-        std::thread::sleep(std::time::Duration::from_millis(50));
+        std::thread::sleep(std::time::Duration::from_millis(10));
         for event in PENDING_QUEUE.lock().drain(..) {
-            sender_clone.send(event).ok();
+            if let Err(e) = sender_clone.send(event) {
+                eprintln!("Warning: Failed to send pending lifecycle event {:?}", e);
+            }
         }
     });
 
@@ -41,22 +44,36 @@ pub fn unsubscribe_runtime() {
 
 pub fn send_lifecycle_event(event: LifecycleEvent) {
     if let Some(sender) = SIGNAL_HANDLER_CHANNEL.lock().as_ref() {
-        sender.send(event).ok();
+        if let Err(e) = sender.send(event) {
+            // Channel full or receivers dropped - this is a critical error for shutdown signals
+            eprintln!("Critical: Failed to send lifecycle event {:?}", e);
+            // For shutdown events, try to force exit if channel delivery fails
+            if matches!(
+                e.0,
+                LifecycleEvent::Shutdown | LifecycleEvent::ForceShutdown
+            ) {
+                eprintln!("Emergency shutdown due to signal delivery failure");
+                std::process::exit(1);
+            }
+        }
     } else {
         PENDING_QUEUE.lock().push_back(event);
     }
 }
 
 fn receive_signal(signum: i32, _: sighandler_t) {
-    SIGINT_COUNT.fetch_add(-1, std::sync::atomic::Ordering::SeqCst);
+    debug!("Received signal: {}", signum);
+    SIGINT_COUNT.fetch_add(-1, Ordering::SeqCst);
     let event = match signum {
         libc::SIGTERM | libc::SIGINT => {
-            SHUTDOWN_REQUESTED.store(true, std::sync::atomic::Ordering::SeqCst);
-            SIGINT_COUNT.fetch_add(2, std::sync::atomic::Ordering::SeqCst);
-            if SIGINT_COUNT.load(std::sync::atomic::Ordering::SeqCst) < 2 {
+            debug!("Received shutdown signal (SIGTERM/SIGINT)");
+            SHUTDOWN_REQUESTED.store(true, Ordering::SeqCst);
+            SIGINT_COUNT.fetch_add(2, Ordering::SeqCst);
+            if SIGINT_COUNT.load(Ordering::SeqCst) < 2 {
+                debug!("First shutdown signal, requesting graceful shutdown");
                 Some(LifecycleEvent::Shutdown)
             } else {
-                // Not messing about. Force shutdown.
+                warn!("Multiple shutdown signals received, forcing immediate shutdown");
                 Some(LifecycleEvent::ForceShutdown)
             }
         }
@@ -70,13 +87,17 @@ fn receive_signal(signum: i32, _: sighandler_t) {
     };
 
     if let Some(event) = event {
+        debug!("Signal {} mapped to lifecycle event: {:?}", signum, event);
         send_lifecycle_event(event);
+    } else {
+        debug!("Signal {} not mapped to any lifecycle event", signum);
     }
 }
 
 pub fn reset_signal_handlers() -> bool {
-    SIGINT_COUNT.store(0, std::sync::atomic::Ordering::SeqCst);
-    SHUTDOWN_REQUESTED.store(false, std::sync::atomic::Ordering::SeqCst);
+    debug!("Resetting signal handlers");
+    SIGINT_COUNT.store(0, Ordering::SeqCst);
+    SHUTDOWN_REQUESTED.store(false, Ordering::SeqCst);
 
     unsafe {
         libc::signal(libc::SIGTERM, receive_signal as usize);
@@ -92,6 +113,7 @@ pub fn reset_signal_handlers() -> bool {
 }
 
 pub fn clear_signal_handlers() {
+    debug!("Clearing signal handlers");
     unsafe {
         libc::signal(libc::SIGTERM, libc::SIG_DFL);
         libc::signal(libc::SIGINT, libc::SIG_DFL);

data/examples/api_with_schema_and_controllers/README.md

@@ -2,7 +2,7 @@
 Simple example of how you can use the following options:
 
 * https://itsi.fyi/middleware/endpoint/
-* https://itsi.fyi/middleware/endpoint/controller
+* https://itsi.fyi/middleware/controller
 * https://itsi.fyi/middleware/endpoint/schemas/
 
 to create validated JSON endpoints directly inside Itsi without a dedicated framework

data/gems/scheduler/Cargo.lock

@@ -213,7 +213,7 @@ checksum = "4a5f13b858c8d314ee3e8f639011f7ccefe71f97f96e50151fb991f267928e2c"
 
 [[package]]
 name = "itsi-scheduler"
-version = "0.2.18"
+version = "0.2.19"
 dependencies = [
  "bytes",
  "derive_more",

data/gems/scheduler/lib/itsi/scheduler/version.rb

@@ -2,6 +2,6 @@
 
 module Itsi
   class Scheduler
-    VERSION = "0.2.18"
+    VERSION = "0.2.19"
   end
 end

data/gems/server/Cargo.lock

@@ -1644,7 +1644,7 @@ checksum = "4a5f13b858c8d314ee3e8f639011f7ccefe71f97f96e50151fb991f267928e2c"
 
 [[package]]
 name = "itsi-server"
-version = "0.2.18"
+version = "0.2.19"
 dependencies = [
  "argon2",
  "async-channel",

data/gems/server/lib/itsi/server/config/middleware/rate_limit.md

@@ -39,7 +39,7 @@ How to identify the client:
   ```
   or
   ```ruby
-  key: { parameter: { query:  { name: "user_id" } } }
+  key: { parameter: { query: "user_id" } }
   ```
 
 ### `store_config`

data/gems/server/lib/itsi/server/version.rb

@@ -2,6 +2,6 @@
 
 module Itsi
   class Server
-    VERSION = "0.2.18"
+    VERSION = "0.2.19"
   end
 end

data/lib/itsi/version.rb

@@ -1,3 +1,3 @@
 module Itsi
-  VERSION = '0.2.18'
+  VERSION = '0.2.19'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: itsi
 version: !ruby/object:Gem::Version
-  version: 0.2.18
+  version: 0.2.19
 platform: ruby
 authors:
 - Wouter Coppieters
@@ -15,28 +15,28 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.2.18
+        version: 0.2.19
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.2.18
+        version: 0.2.19
 - !ruby/object:Gem::Dependency
   name: itsi-server
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.2.18
+        version: 0.2.19
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.2.18
+        version: 0.2.19
 description: Wrapper Gem for both the Itsi server and the Itsi Fiber scheduler
 email:
 - wc@pico.net.nz