Merge pull request #419 from chadlwilson/1.2-safe-jruby-10

1.2: Backport JRuby 10.0 and Rails 8.0 compatbility

Author: chadlwilson

.github/workflows/maven.yml

@@ -19,9 +19,16 @@ jobs:
 
     strategy:
       matrix:
-        jruby_version: [ '9.3.15.0', '9.4.14.0' ]
+        jruby_version: [ '9.3.15.0', '9.4.14.0', '10.0.5.0' ]
         java_version: [ '8', '11', '17', '21', '25' ]
         rack_version: [ '~> 2.2.0' ]
+        exclude:
+          - jruby_version: '10.0.5.0'
+            java_version: '8' # JRuby 10 requires Java 21
+          - jruby_version: '10.0.5.0'
+            java_version: '11' # JRuby 10 requires Java 21
+          - jruby_version: '10.0.5.0'
+            java_version: '17' # JRuby 10 requires Java 21
       fail-fast: false
 
     steps:
@@ -59,16 +66,27 @@ jobs:
           'rails70_rack22',
           'rails71_rack22',
           'rails72_rack22',
+          'rails80_rack22',
         ]
-        jruby_version: [ '9.3.15.0', '9.4.14.0' ]
+        jruby_version: [ '9.3.15.0', '9.4.14.0', '10.0.5.0' ]
         java_version: [ '8', '11', '17', '21', '25' ]
         exclude:
+          - jruby_version: '10.0.5.0'
+            java_version: '8' # JRuby 10 requires Java 21
+          - jruby_version: '10.0.5.0'
+            java_version: '11' # JRuby 10 requires Java 21
+          - jruby_version: '10.0.5.0'
+            java_version: '17' # JRuby 10 requires Java 21
           - appraisal: 'rails70_rack22' # Requires Ruby 2.7 compatibility, which JRuby 9.3 does not support
             jruby_version: '9.3.15.0'
           - appraisal: 'rails71_rack22' # Requires Ruby 2.7 compatibility, which JRuby 9.3 does not support
             jruby_version: '9.3.15.0'
           - appraisal: 'rails72_rack22' # Requires Ruby 3.1 compatibility, which JRuby 9.3 does not support
             jruby_version: '9.3.15.0'
+          - appraisal: 'rails80_rack22' # Requires Ruby 3.4 compatibility, which JRuby 9.3 does not support
+            jruby_version: '9.3.15.0'
+          - appraisal: 'rails80_rack22' # Requires Ruby 3.4 compatibility, which JRuby 9.4 does not support
+            jruby_version: '9.4.14.0'
       fail-fast: false
 
     env:

Appraisals

@@ -1,22 +1,26 @@
 version_spec = ->(prefix, desc) { "~> #{desc.split(prefix).last.insert(1, ".")}.0" }
 
-# Rails version -> rack versions in format
-# rails#{MAJOR}#{MINOR} => %w[ rack#{MAJOR}#{MINOR} ]
+# rails#{MAJOR}#{MINOR} => config_obj
 {
-    "rails50" => %w[rack22],
-    "rails52" => %w[rack22],
-    "rails60" => %w[rack22],
-    "rails61" => %w[rack22],
-    "rails70" => %w[rack22],
-    "rails71" => %w[rack22],
-    "rails72" => %w[rack22]
-}.each do |rails_desc, rack_descs|
-  rack_descs.each do |rack_desc|
+  "rails50" => {racks: %w[rack22], ext_gems: %w[mutex_m bigdecimal base64]},
+  "rails52" => {racks: %w[rack22], ext_gems: %w[mutex_m bigdecimal]},
+  "rails60" => {racks: %w[rack22], ext_gems: %w[mutex_m bigdecimal]},
+  "rails61" => {racks: %w[rack22], ext_gems: %w[mutex_m bigdecimal]},
+  "rails70" => {racks: %w[rack22]},
+  "rails71" => {racks: %w[rack22]},
+  "rails72" => {racks: %w[rack22]},
+  "rails80" => {racks: %w[rack22]},
+}.each do |rails_desc, c|
+  c[:racks].each do |rack_desc|
 
     appraise "#{rails_desc}_#{rack_desc}" do
       group :default do
         gem "rack", version_spec.call("rack", rack_desc)
         gem "rails", version_spec.call("rails", rails_desc)
+
+        c[:ext_gems]&.each do |gem_name|
+          gem gem_name
+        end
       end
     end
   end

CHANGELOG.md

@@ -1,5 +1,6 @@
 ## 1.2.7 (UNRELEASED)
 
+- Fix compatibility with JRuby 10.0 and Rails 8.0 (#419)
 - Fix ability to include and forward to JSPs under Rails (#370)
 - Update (bundled) rack to 2.2.23 (#417)
 

README.md

@@ -13,11 +13,11 @@ For more information on Rack, visit http://rack.github.io/.
 
 ## Compatibility
 
-| JRuby-Rack Series                                              | Status        | Rack      | JRuby     | Java | Rails     | Target Servlet API | Notes                                                          |
-|----------------------------------------------------------------|---------------|-----------|-----------|------|-----------|--------------------|----------------------------------------------------------------|
-| [**1.2**](https://github.com/jruby/jruby-rack/tree/1.2-stable) | Maintained    | 2.2       | 9.3 → 9.4 | 8+   | 5.0 → 7.2 | 3.0 (Java EE 6)    | ✅ _Unofficial_: Servlet 3.1 → 4.0 also OK with most containers |
-| [**1.1**](https://github.com/jruby/jruby-rack/tree/1.1-stable) | EOL @ 2024-05 | 1.x → 2.2 | 1.6 → 9.4 | 6+   | 2.1 → 5.2 | 2.5 (Java EE 5)    | ✅ _Unofficial_: Servlet 3.0 → 4.0 also OK with most containers |
-| [**1.0**](https://github.com/jruby/jruby-rack/tree/1.0.10)     | EOL @ 2011-11 | 0.9 → 1.x | 1.1 → 1.9 | 5+   | 2.1 → 3.x | 2.5 (Java EE 5)    |                                                                |
+| JRuby-Rack Series                                              | Status        | Rack      | JRuby      | Java | Rails     | Target Servlet API | Notes                                                          |
+|----------------------------------------------------------------|---------------|-----------|------------|------|-----------|--------------------|----------------------------------------------------------------|
+| [**1.2**](https://github.com/jruby/jruby-rack/tree/1.2-stable) | Maintained    | 2.2       | 9.3 → 10.0 | 8+   | 5.0 → 8.0 | 3.0 (Java EE 6)    | ✅ _Unofficial_: Servlet 3.1 → 4.0 also OK with most containers |
+| [**1.1**](https://github.com/jruby/jruby-rack/tree/1.1-stable) | EOL @ 2024-05 | 1.x → 2.2 | 1.6 → 9.4  | 6+   | 2.1 → 5.2 | 2.5 (Java EE 5)    | ✅ _Unofficial_: Servlet 3.0 → 4.0 also OK with most containers |
+| [**1.0**](https://github.com/jruby/jruby-rack/tree/1.0.10)     | EOL @ 2011-11 | 0.9 → 1.x | 1.1 → 1.9  | 5+   | 2.1 → 3.x | 2.5 (Java EE 5)    |                                                                |
 
 ## Getting Started
 
@@ -209,8 +209,6 @@ as context init parameters in web.xml or as VM-wide system properties.
   sub-path of the main servlet context root.
 - `gem.path`: Relative path to the bundled gem repository. Defaults to
   `/WEB-INF/gems`.
-- `jruby.compat.version`: Set to "1.8" or "1.9" to make JRuby run a specific
-  version of Ruby (same as the --1.8 / --1.9 command line flags).
 - `jruby.min.runtimes`: For non-threadsafe Rails applications using a runtime
   pool, specify an integer minimum number of runtimes to hold in the pool.
 - `jruby.max.runtimes`: For non-threadsafe Rails applications, an integer

gemfiles/rails50_rack22.gemfile

@@ -8,6 +8,9 @@ gem "rspec", group: :test
 group :default do
   gem "rack", "~> 2.2.0"
   gem "rails", "~> 5.0.0"
+  gem "mutex_m"
+  gem "bigdecimal"
+  gem "base64"
 end
 
 group :development do

gemfiles/rails52_rack22.gemfile

@@ -8,6 +8,8 @@ gem "rspec", group: :test
 group :default do
   gem "rack", "~> 2.2.0"
   gem "rails", "~> 5.2.0"
+  gem "mutex_m"
+  gem "bigdecimal"
 end
 
 group :development do

gemfiles/rails60_rack22.gemfile

@@ -8,6 +8,8 @@ gem "rspec", group: :test
 group :default do
   gem "rack", "~> 2.2.0"
   gem "rails", "~> 6.0.0"
+  gem "mutex_m"
+  gem "bigdecimal"
 end
 
 group :development do

gemfiles/rails61_rack22.gemfile

@@ -8,6 +8,8 @@ gem "rspec", group: :test
 group :default do
   gem "rack", "~> 2.2.0"
   gem "rails", "~> 6.1.0"
+  gem "mutex_m"
+  gem "bigdecimal"
 end
 
 group :development do

gemfiles/rails80_rack22.gemfile

@@ -0,0 +1,15 @@
+# This file was generated by Appraisal
+
+source "https://rubygems.org"
+
+gem "rake", "~> 13.3", group: :test, require: nil
+gem "rspec", group: :test
+
+group :default do
+  gem "rack", "~> 2.2.0"
+  gem "rails", "~> 8.0.0"
+end
+
+group :development do
+  gem "appraisal", require: nil
+end

src/main/java/org/jruby/rack/DefaultRackApplicationFactory.java

@@ -302,10 +302,6 @@ protected RubyInstanceConfig initRuntimeConfig(final RubyInstanceConfig config)
         // Process arguments, namely any that might be in RUBYOPT
         config.processArguments(rackConfig.getRuntimeArguments());
 
-        if ( rackConfig.getCompatVersion() != null ) {
-            config.setCompatVersion(rackConfig.getCompatVersion());
-        }
-
         try { // try to set jruby home to jar file path
             final URL resource = Ruby.class.getResource("/META-INF/jruby.home");
             if ( resource != null && "jar".equals( resource.getProtocol() ) ) {
@@ -487,7 +483,7 @@ private void captureMessage(final RaiseException re) {
             rubyException.callMethod(context, "capture");
             rubyException.callMethod(context, "store");
         }
-        catch (Exception e) {
+        catch (Throwable e) {
             rackContext.log(INFO, "failed to capture exception message", e);
             // won't be able to capture anything
         }