rails_twirp 0.15 → 0.16

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4e1b6e06cf246cc4ab4a639ca805033c8f96a24d2a1f468b733001b5f4ca9732
-  data.tar.gz: 4e03e912af4c785bbe9852245ed6e763e6d398c335dd63d4d0c4c3751402b814
+  metadata.gz: 49a7d34d89a4dac62fb2125561c1b7aa3dc4fcbe429c13eb14d8cadc04d5ba39
+  data.tar.gz: '0491bab9e0a82bc44551047c4a1fe5969903e168e66cd1ab98635277a8e94013'
 SHA512:
-  metadata.gz: 6cdc2a1d80937bbb85a513cc4807e9a91c5bd0848d5b959e90d226477231387c0e9250ec30bf7ff8da170c8cf64ca7136bcd47deb3fb23206797749bdbe624b8
-  data.tar.gz: 68c342ad35489011bc921dc4c7529d5ff7a18f47fce4aef9c597cebd2a789bdcec8050b6738cad457bcb54fce0b1c9f262558a964d42b26e1063a34f73f9d250
+  metadata.gz: 8320ba3215e7cdea0cf333611f3df86a5a134f8c2d61bac51b75a04cccd3f644f5b4cd690239ce4a4b42ad22429de88aacbfb5b40e8da67844145806625d675f
+  data.tar.gz: 0d6a125a57a4b709702c54db8d6ed1339d6f89f6c5e898b12136fb0ec1de3cb8ceeeede9bb9a040193240922fe00666aa5205b50c7075f118d989993c10d9620

data/.standard.yml

@@ -1,3 +1 @@
-ignore:
-  - '**/*':
-    - Lint/RescueException
+ruby_version: 2.7

data/CHANGELOG.md

@@ -1,9 +1,13 @@
-### 0.15.0
+### 0.16
+
+* Ensure `decode_rack_response` always calls `#close` on the Rack body in tests, as some middleware might be applying a BodyProxy
+
+### 0.15
 
 * Exclude versions of Rails 7 which were incompatible with the pbbuilder ActionView handler, as pbbuilder cannot work there at all
 * Fix decode_rack_response to be compatible with Rack response body wrappers (and conform to the Rack SPEC)
 
-### 0.14.0
+### 0.14
 * Adding frozen_string_literal: true to all files.
 
 ### 0.13.2
@@ -18,14 +22,14 @@
 * Include `ActionController::Caching` with Base controller/helpers
 
 
-### 0.12.0
+### 0.12
 
 * Allow a custom exception handling proc to be assigned using `RailsTwirp.unhandled_exception_handler`
 
-### 0.11.0
+### 0.11
 
 * Update configuration and tests for Rails 7 compatibility
 
-### 0.10.0
+### 0.10
 
 * Handle exceptions more like the Rails controllers do it, do not capture all of them as if they were Twirp exceptions

data/lib/rails_twirp/exception_handling.rb

@@ -10,7 +10,7 @@ module RailsTwirp
 
     def process_action(*)
       super
-    rescue Exception => e
+    rescue => e
       # Only the exceptions which are not captured by ActionController-like "rescue_from" end up here.
       # The idea is that any exception which is rescued by the controller is treated as part of the business
       # logic, and thus taking action on it is the responsibility of the controller which uses "rescue_from".

data/lib/rails_twirp/mapper.rb

@@ -59,10 +59,10 @@ module RailsTwirp
       @module = nil
     end
 
-    def service(service_definition, **, &block)
+    def service(service_definition, **kwargs, &block)
       service_route_set = @route_set.services[service_definition]
       service_mapper = ServiceMapper.new(service_route_set, self)
-      scope(**) { service_mapper.instance_exec(&block) }
+      scope(**kwargs) { service_mapper.instance_exec(&block) }
     end
 
     def scope(**options)

data/lib/rails_twirp/rescue.rb

@@ -9,8 +9,8 @@ module RailsTwirp
 
     def process_action(*)
       super
-    rescue Exception => exception
-      rescue_with_handler(exception) || raise
+    rescue => e
+      rescue_with_handler(e) || raise
     end
   end
 end

data/lib/rails_twirp/testing/integration_test.rb

@@ -91,14 +91,17 @@ module RailsTwirp
     end
 
     def decode_rack_response(service, rpc, status, headers, body)
-      body = Array.wrap(body).join # body is each-able
+      body_bytes = StringIO.new("".b)
+      body.each { |b| body_bytes << b }
 
       if status === 200
         output_class = service.rpcs[rpc][:output_class]
-        Twirp::Encoding.decode(body, output_class, headers["Content-Type"])
+        Twirp::Encoding.decode(body_bytes.string, output_class, headers["Content-Type"])
       else
-        Twirp::Client.error_from_response(Response.new(status, body, headers))
+        Twirp::Client.error_from_response(Response.new(status, body_bytes.string, headers))
       end
+    ensure
+      body.close if body.respond_to?(:close) # Comply with Rack API
     end
 
     def set_controller_from_rack_env(env)

data/lib/rails_twirp/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module RailsTwirp
-  VERSION = "0.15"
+  VERSION = "0.16"
 end

data/rails_twirp.gemspec

@@ -13,10 +13,10 @@ Gem::Specification.new do |spec|
 
   spec.files = `git ls-files`.split("\n")
 
-  # Rails has shipped an incompatible change in ActiveView, that was reverted in later versions.
+  # Rails has shipped an incompatible change in ActiveView, that was reverted in later versions
+  # but at this time has not been released as a 7.x version
   # @see https://github.com/rails/rails/pull/51023
-  excluded_versions = ["7.1.0", "7.1.1", "7.1.2", "7.1.3"].map { |v| "!= #{v}" }
-  spec.add_runtime_dependency "rails", ">= 6.1.3", *excluded_versions
+  spec.add_runtime_dependency "rails", ">= 6.1.3", " < 7.1"
   spec.add_runtime_dependency "twirp", ">= 1.9", "< 1.11"
   spec.required_ruby_version = ">= 3"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rails_twirp
 version: !ruby/object:Gem::Version
-  version: '0.15'
+  version: '0.16'
 platform: ruby
 authors:
 - Bouke van der Bijl
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-02-13 00:00:00.000000000 Z
+date: 2024-03-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -17,18 +17,9 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 6.1.3
-    - - "!="
-      - !ruby/object:Gem::Version
-        version: 7.1.0
-    - - "!="
-      - !ruby/object:Gem::Version
-        version: 7.1.1
-    - - "!="
-      - !ruby/object:Gem::Version
-        version: 7.1.2
-    - - "!="
+    - - "<"
       - !ruby/object:Gem::Version
-        version: 7.1.3
+        version: '7.1'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -36,18 +27,9 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 6.1.3
-    - - "!="
-      - !ruby/object:Gem::Version
-        version: 7.1.0
-    - - "!="
-      - !ruby/object:Gem::Version
-        version: 7.1.1
-    - - "!="
-      - !ruby/object:Gem::Version
-        version: 7.1.2
-    - - "!="
+    - - "<"
       - !ruby/object:Gem::Version
-        version: 7.1.3
+        version: '7.1'
 - !ruby/object:Gem::Dependency
   name: twirp
   requirement: !ruby/object:Gem::Requirement