message_bus 4.4.1 → 4.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c4a55e419b6f5807f46429d27360a776c8c8057d663188f1911aca78cb6198fc
-  data.tar.gz: dd96e8f87e10aca16262ca540997011871f0ba809a1153eb330aa82d8683267d
+  metadata.gz: c0191a3cefe09aba3cbad0048cb6dd56ed1e9d05159ac66f26429715f3e785d6
+  data.tar.gz: bcdcb1b72f53eb15c220ca4909de5ceadb251678a174d16f3e277934bf72802d
 SHA512:
-  metadata.gz: b62eacf519bc544cde94ac05683938c2ca4c98bfaeaefd79e99ac012864d4c0ac20412ccc9fd0910cdf122b2883841af24339d299101b35e7296c0faa60c103f
-  data.tar.gz: 44fb3cda5e137b61f35d5b0f1be204ab179594df59423948946ad4790b5b9bf3d8087be3aa55f819dfe9df6b65d3a30138312cc69a0726711bd65821789f459c
+  metadata.gz: 47ee3632a6eca276a1eacb3ddde8f5e56b36d11d30ea8414756338e6a0baec6f6d609cc7a1bc2c9d86561d6d9f1c2347a2e1f912cf3a519fb1c842b9e83aab06
+  data.tar.gz: da53d5bb3626d27f9b8236c31064496f085c09486149d584b17aee86ac7ed58d1809c9b8c4f7151fe986b9d83dd89d26761cc878b2e6feb27de3d6b4c1de7530

data/.github/workflows/ci.yml

@@ -19,20 +19,22 @@ jobs:
           ruby-version: "3.3"
           bundler-cache: true
 
-      - name: Set up Node.js
-        uses: actions/setup-node@v4
+      - uses: pnpm/action-setup@v4
+
+      - uses: actions/setup-node@v6
         with:
-          node-version: 20
-          cache: npm
+          node-version: "24"
+          cache: "pnpm"
 
-      - name: Setup npm
-        run: npm install
+      - name: Pnpm install
+        run: pnpm install
 
       - name: Rubocop
         run: bundle exec rubocop
 
       - name: ESLint
-        run: npx eslint .
+        if: ${{ !cancelled() }}
+        run: pnpm lint
 
   test:
     runs-on: ubuntu-latest
@@ -77,6 +79,16 @@ jobs:
           ruby-version: ${{ matrix.ruby }}
           bundler-cache: true
 
+      - uses: pnpm/action-setup@v4
+
+      - uses: actions/setup-node@v6
+        with:
+          node-version: "24"
+          cache: "pnpm"
+
+      - name: Pnpm install
+        run: pnpm install
+
       - name: Tests
         env:
           TESTOPTS: --verbose
@@ -87,6 +99,10 @@ jobs:
     needs: [lint, test]
     runs-on: ubuntu-latest
 
+    permissions:
+      id-token: write
+      contents: write
+
     steps:
       - uses: actions/checkout@v4
 
@@ -108,8 +124,13 @@ jobs:
           git add package.json
           git commit -m 'bump'
 
+      - uses: pnpm/action-setup@v4
+
+      - uses: actions/setup-node@v6
+        with:
+          node-version: "24"
+          cache: "pnpm"
+
       - name: Publish package
-        uses: JS-DevTools/npm-publish@v3
         if: steps.publish-gem.outputs.new_version == 'true'
-        with:
-          token: ${{ secrets.NPM_TOKEN }}
+        run: pnpm publish -r

data/.gitignore

@@ -16,7 +16,5 @@ test/tmp
 test/version_tmp
 tmp
 *.swp
-.rubocop-https---raw-githubusercontent-com-discourse-discourse-master--rubocop-yml
 .byebug_history
 node_modules/
-yarn.lock

data/CHANGELOG

@@ -1,3 +1,10 @@
+10-02-2026
+
+- Version 4.5.0
+
+  - Support chunked encoding for Puma 7 on Rack 3.1
+  - FIX: Prune empty subscriptions in `ConnectionManager#remove_client`
+
 20-03-2025
 
 - Version 4.4.1

data/README.md

@@ -4,7 +4,7 @@ A reliable, robust messaging bus for Ruby processes and web clients.
 
 MessageBus implements a Server to Server channel based protocol and Server to Web Client protocol (using polling, long-polling or long-polling + streaming)
 
-Since long-polling is implemented using Rack Hijack and Thin::Async, all common Ruby web servers (Thin, Puma, Unicorn, Passenger) can run MessageBus and handle a large number of concurrent connections that wait on messages.
+Since long-polling is implemented using Rack Hijack and Thin::Async, all common Ruby web servers (Pitchfork/Unicorn, Puma, Thin, Passenger) can run MessageBus and handle a large number of concurrent connections that wait on messages.
 
 MessageBus is implemented as Rack middleware and can be used by any Rails / Sinatra or pure Rack application.
 
@@ -381,7 +381,9 @@ MessageBus.configure(keepalive_interval: 60)
 message_bus supports using Redis as a storage backend, and you can configure message_bus to use redis in `config/initializers/message_bus.rb`, like so:
 
 ```ruby
-MessageBus.configure(backend: :redis, url: "redis://:p4ssw0rd@10.0.1.1:6380/15")
+MessageBus.configure(backend: :redis, redis_config: { 
+  url: "redis://:p4ssw0rd@10.0.1.1:6380/15"
+})
 ```
 
 The redis client message_bus uses is [redis-rb](https://github.com/redis/redis-rb), so you can visit it's repo to see what other options you can pass besides a `url`.
@@ -502,7 +504,7 @@ on_worker_boot do
 end
 ```
 
-#### Unicorn
+#### Pitchfork/Unicorn
 
 ```ruby
 # path/to/your/config/unicorn.rb

data/Rakefile

@@ -35,7 +35,7 @@ end
 namespace :jasmine do
   desc "Run Jasmine tests in headless mode"
   task 'ci' do
-    if !system("npx jasmine-browser-runner runSpecs")
+    if !system("pnpm jasmine-browser-runner runSpecs")
       exit 1
     end
   end
@@ -71,7 +71,7 @@ namespace :spec do
 
     begin
       ENV['MESSAGE_BUS_BACKEND'] = 'memory'
-      pid = spawn("bundle exec puma -p 9292 spec/fixtures/test/config.ru")
+      pid = spawn("bundle exec pitchfork -p 9292 spec/fixtures/test/config.ru")
       sleep 1 while port_available?(9292)
       Rake::TestTask.new(:integration) do |t|
         t.test_files = INTEGRATION_FILES

data/docker-compose.yml

@@ -36,7 +36,7 @@ services:
   example:
     build:
       context: .
-    command: bash -c "cd examples/chat && bundle install && bundle exec rackup --server puma --host 0.0.0.0"
+    command: bash -c "cd examples/chat && bundle install && bundle exec rackup --server pitchfork --host 0.0.0.0"
     environment:
       BUNDLE_TO: /usr/local/bundle
       REDISURL: redis://redis:6379

data/examples/bench/config.ru

@@ -30,4 +30,4 @@ end
 MessageBus.long_polling_interval = 1000 * 2
 MessageBus.max_active_clients = 10000
 use MessageBus::Rack::Middleware
-run lambda { |_env| [200, { "Content-Type" => "text/html" }, ["Howdy"]] }
+run lambda { |_env| [200, { "content-type" => "text/html" }, ["Howdy"]] }

data/examples/chat/Gemfile

@@ -1,4 +1,4 @@
 # frozen_string_literal: true
-gem 'puma'
+gem 'pitchfork'
 gem 'redis'
 gem 'sinatra'

data/examples/chat/docker_container/chat.yml

@@ -33,18 +33,18 @@ hooks:
     - exec: cd /var/www && git clone --depth 1 https://github.com/SamSaffron/message_bus.git
     - exec:
         cmd:
-          - gem install puma
+          - gem install pitchfork
           - gem install redis
           - gem install sinatra
     - file:
-       path: /etc/service/puma/run
+       path: /etc/service/pitchfork/run
        chmod: "+x"
        contents: |
          #!/bin/bash
          exec 2>&1
          # redis
          cd $home/examples/chat
-         exec sudo -E -u chat LD_PRELOAD=/usr/lib/libjemalloc.so.1 puma -p 8080 -e production
+         exec sudo -E -u chat LD_PRELOAD=/usr/lib/libjemalloc.so.1 pitchfork -p 8080 -e production
     - exec: rm /etc/nginx/sites-enabled/default
     - replace:
         filename: /etc/nginx/nginx.conf

data/examples/minimal/config.ru

@@ -4,4 +4,4 @@ require 'message_bus'
 # MessageBus.long_polling_interval = 1000 * 2
 
 use MessageBus::Rack::Middleware
-run lambda { |_env| [200, { "Content-Type" => "text/html" }, ["Howdy"]] }
+run lambda { |_env| [200, { "content-type" => "text/html" }, ["Howdy"]] }

data/lib/message_bus/connection_manager.rb

@@ -76,8 +76,13 @@ class MessageBus::ConnectionManager
   def remove_client(c)
     synchronize do
       @clients.delete c.client_id
-      @subscriptions[c.site_id].each do |_k, set|
-        set.delete c.client_id
+      site_subs = @subscriptions[c.site_id]
+      if site_subs
+        site_subs.delete_if do |_k, set|
+          set.delete c.client_id
+          set.empty?
+        end
+        @subscriptions.delete(c.site_id) if site_subs.empty?
       end
       if c.cleanup_timer
         # concurrency may cause this to fail

data/lib/message_bus/rack/middleware.rb

@@ -82,10 +82,10 @@ class MessageBus::Rack::Middleware
     return [404, {}, ["not found"]] unless client_id
 
     headers = {}
-    headers["Cache-Control"] = "must-revalidate, private, max-age=0"
-    headers["Content-Type"] = "application/json; charset=utf-8"
-    headers["Pragma"] = "no-cache"
-    headers["Expires"] = "0"
+    headers["cache-control"] = "must-revalidate, private, max-age=0"
+    headers["content-type"] = "application/json; charset=utf-8"
+    headers["pragma"] = "no-cache"
+    headers["expires"] = "0"
 
     if @bus.extra_response_headers_lookup
       @bus.extra_response_headers_lookup.call(env).each do |k, v|
@@ -131,7 +131,7 @@ class MessageBus::Rack::Middleware
                    env['QUERY_STRING'] !~ /dlp=t/ &&
                    @connection_manager.client_count < @bus.max_active_clients
 
-    allow_chunked = env['HTTP_VERSION'] == 'HTTP/1.1'
+    allow_chunked = env['SERVER_PROTOCOL'] == 'HTTP/1.1' || env['HTTP_VERSION'] == 'HTTP/1.1'
     allow_chunked &&= !env['HTTP_DONT_CHUNK']
     allow_chunked &&= @bus.chunked_encoding_enabled?
 

data/lib/message_bus/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module MessageBus
-  VERSION = "4.4.1"
+  VERSION = "4.5.0"
 end

data/message_bus.gemspec

@@ -16,7 +16,7 @@ Gem::Specification.new do |gem|
   gem.version       = MessageBus::VERSION
   gem.required_ruby_version = ">= 2.6.0"
 
-  gem.add_runtime_dependency 'rack', '>= 1.1.3'
+  gem.add_runtime_dependency 'rack', '~> 3'
 
   # Optional runtime dependencies
   gem.add_development_dependency 'redis'
@@ -30,9 +30,10 @@ Gem::Specification.new do |gem|
   gem.add_development_dependency 'http_parser.rb'
   gem.add_development_dependency 'thin'
   gem.add_development_dependency 'rack-test'
-  gem.add_development_dependency 'puma'
+  gem.add_development_dependency 'pitchfork'
   gem.add_development_dependency 'm'
   gem.add_development_dependency 'byebug'
+  gem.add_development_dependency 'method_source'
   gem.add_development_dependency 'oj'
   gem.add_development_dependency 'yard'
 

data/package.json

@@ -19,9 +19,16 @@
     "url": "https://github.com/discourse/message_bus/issues"
   },
   "homepage": "https://github.com/discourse/message_bus#readme",
+  "publishConfig": {
+    "access": "public"
+  },
+  "scripts": {
+    "lint": "eslint ."
+  },
   "devDependencies": {
     "eslint": "^8.31.0",
     "jasmine-browser-runner": "^0.10.0",
     "jasmine-core": "^3.10.1"
-  }
+  },
+  "packageManager": "pnpm@9.15.5"
 }