Fix decode_rack_response

The Rack response body is _not_ an Enumerable, it just responds to #each. The method was assuming otherwise, which led to breakage when trying to use appsignal/appsignal-ruby#1037
cheddar-me · Feb 13, 2024 · df490eb · df490eb
1 parent d8e8c01
commit df490eb
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,7 @@
+### 0.15.0
+
+* Fix decode_rack_response to be compatible with Rack response body wrappers (and conform to the Rack SPEC)
+
 ### 0.14.0
 * Adding frozen_string_literal: true to all files.
 

diff --git a/lib/rails_twirp/testing/integration_test.rb b/lib/rails_twirp/testing/integration_test.rb
@@ -91,7 +91,7 @@ def request_content_type
     end
 
     def decode_rack_response(service, rpc, status, headers, body)
-      body = body.join # body is an Enumerable
+      body = Array.wrap(body).join # body is each-able
 
       if status === 200
         output_class = service.rpcs[rpc][:output_class]

diff --git a/lib/rails_twirp/version.rb b/lib/rails_twirp/version.rb
@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module RailsTwirp
-  VERSION = "0.14"
+  VERSION = "0.15"
 end