message_bus 4.4.1 → 4.5.2

data/spec/fixtures/test/config.ru

@@ -14,5 +14,5 @@ run ->(env) do
     end
   end
 
-  [200, { "Content-Type" => "text/html" }, ["Howdy"]]
+  [200, { "content-type" => "text/html" }, ["Howdy"]]
 end

data/spec/integration/http_client_spec.rb

@@ -65,7 +65,7 @@ describe MessageBus::HTTPClient do
           # that we sleep for the right interval after failure
           sleep 0.5
 
-          assert_match(/Errno::ECONNREFUSED|SocketError/, fake.string)
+          assert_match(/Errno::ECONNREFUSED|SocketError|Socket::ResolutionError/, fake.string)
         ensure
           $stderr = original_stderr
         end

data/spec/lib/message_bus/client_spec.rb

@@ -32,6 +32,8 @@ describe MessageBus::Client do
         break if line == ""
 
         name, val = line.split(": ")
+        name = name.downcase
+        raise "duplicate header: #{name}" if headers.key?(name)
         headers[name] = val
       end
 
@@ -59,7 +61,7 @@ describe MessageBus::Client do
       @client.use_chunked = true
       r, w = IO.pipe
       @client.io = w
-      @client.headers = { "Content-Type" => "application/json; charset=utf-8" }
+      @client.headers = { "content-type" => "application/json; charset=utf-8" }
       @client << MessageBus::Message.new(1, 1, '/test', 'test')
       @client << MessageBus::Message.new(2, 2, '/test', "a|\r\n|\r\n|b")
 
@@ -67,7 +69,7 @@ describe MessageBus::Client do
 
       status, headers, chunks = http_parse(lines)
 
-      headers["Content-Type"].must_equal "text/plain; charset=utf-8"
+      headers["content-type"].must_equal "text/plain; charset=utf-8"
       status.must_equal "200"
       chunks.length.must_equal 2
 

data/spec/lib/message_bus/connection_manager_spec.rb

@@ -90,6 +90,29 @@ describe MessageBus::ConnectionManager do
     @manager.notify_clients(m)
     assert_nil @resp.sent
   end
+
+  it "should prune empty channel and site_id entries from subscriptions after removing clients" do
+    client1 = MessageBus::Client.new(client_id: "abc", user_id: 1, site_id: 20)
+    client1.subscribe('test', -1)
+    @manager.add_client(client1)
+
+    client2 = MessageBus::Client.new(client_id: "def", user_id: 2, site_id: 20)
+    client2.subscribe('test', -1)
+    @manager.add_client(client2)
+
+    @manager.remove_client(client1)
+
+    subscriptions = @manager.instance_variable_get(:@subscriptions)
+    assert subscriptions[client1.site_id].key?('test')
+    assert subscriptions[client1.site_id]['test'].include?(client2.client_id)
+    refute subscriptions[client1.site_id]['test'].include?(client1.client_id)
+
+    @manager.remove_client(client2)
+
+    subscriptions = @manager.instance_variable_get(:@subscriptions)
+    assert_nil subscriptions.dig(client1.site_id, 'test')
+    assert_nil subscriptions[client1.site_id]
+  end
 end
 
 describe MessageBus::ConnectionManager, "notifying and subscribing concurrently" do

data/spec/lib/message_bus/rack/middleware_spec.rb

@@ -21,7 +21,7 @@ describe MessageBus::Rack::Middleware do
       use FakeAsyncMiddleware, message_bus: bus
       use e_m if e_m
       use MessageBus::Rack::Middleware, message_bus: bus
-      run lambda { |_env| [500, { 'Content-Type' => 'text/html' }, 'should not be called'] }
+      run lambda { |_env| [500, { 'content-type' => 'text/html' }, 'should not be called'] }
     }
 
     @async_middleware = builder.to_app

metadata

@@ -1,28 +1,34 @@
 --- !ruby/object:Gem::Specification
 name: message_bus
 version: !ruby/object:Gem::Version
-  version: 4.4.1
+  version: 4.5.2
 platform: ruby
 authors:
 - Sam Saffron
 bindir: bin
 cert_chain: []
-date: 2025-03-20 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - ">"
+      - !ruby/object:Gem::Version
+        version: '2'
+    - - "<"
       - !ruby/object:Gem::Version
-        version: 1.1.3
+        version: '4'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - ">"
       - !ruby/object:Gem::Version
-        version: 1.1.3
+        version: '2'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '4'
 - !ruby/object:Gem::Dependency
   name: redis
   requirement: !ruby/object:Gem::Requirement
@@ -164,7 +170,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: puma
+  name: pitchfork
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -205,6 +211,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: method_source
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: oj
   requirement: !ruby/object:Gem::Requirement
@@ -260,7 +280,6 @@ files:
 - ".prettierrc"
 - ".rubocop.yml"
 - CHANGELOG
-- DEV.md
 - Dockerfile
 - Gemfile
 - Guardfile
@@ -289,6 +308,8 @@ files:
 - examples/chat/docker_container/update_chat
 - examples/minimal/Gemfile
 - examples/minimal/config.ru
+- gemfiles/rack2.gemfile
+- gemfiles/rack3.gemfile
 - lib/message_bus.rb
 - lib/message_bus/backends.rb
 - lib/message_bus/backends/base.rb
@@ -311,8 +332,8 @@ files:
 - lib/message_bus/timer_thread.rb
 - lib/message_bus/version.rb
 - message_bus.gemspec
-- package-lock.json
 - package.json
+- pnpm-lock.yaml
 - spec/assets/SpecHelper.js
 - spec/assets/message-bus.spec.js
 - spec/fixtures/test/Gemfile
@@ -347,14 +368,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.6.0
+      version: 3.2.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.6.2
+rubygems_version: 3.6.9
 specification_version: 4
 summary: ''
 test_files: []

data/DEV.md

@@ -1,7 +0,0 @@
-### How to Publish to NPM
-
-1. First, edit `package.json` and bump the version.
-
-2. Log in to npm `yarn login`
-
-3. Publish: `yarn publish`