From 966e39174d28747a4d578e2520c4e51bb39e0005 Mon Sep 17 00:00:00 2001 From: Myron Marston Date: Mon, 4 Nov 2024 11:02:34 -0800 Subject: [PATCH] Deal with rackup 2.2 release. Rackup 2.2 no longer depends on `webrick`, and our usage of `rackup` fails if we don't add a dependency on `webrick`. For details on the `rackup` change, see: https://github.com/rack/rackup/pull/23 Note that the `rackup` maintainers recommend using `falcon` or `puma` instead of `webrick`. However, `falcon` and `puma` are heavier weight. `falcon` has lots of runtime dependencies and `puma` depends on `nio4r` which has native code. As a result, it's more likely to hit installation issues for ElasticGraph users than `webrick`. So for now we are sticking with `webrick` but I'm open to changing this in the future. --- elasticgraph-apollo/apollo_tests_implementation/Gemfile | 3 ++- elasticgraph-local/elasticgraph-local.gemspec | 3 ++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/elasticgraph-apollo/apollo_tests_implementation/Gemfile b/elasticgraph-apollo/apollo_tests_implementation/Gemfile index 0b43ce65..51b39f5d 100644 --- a/elasticgraph-apollo/apollo_tests_implementation/Gemfile +++ b/elasticgraph-apollo/apollo_tests_implementation/Gemfile @@ -24,4 +24,5 @@ source "https://rubygems.org" gem "elasticgraph-#{suffix}", path: "elasticgraph-#{suffix}" end -gem "rackup", "~> 2.1" +gem "rackup", "~> 2.2" +gem "webrick", "~> 1.9" diff --git a/elasticgraph-local/elasticgraph-local.gemspec b/elasticgraph-local/elasticgraph-local.gemspec index 661aacf3..c96d4c50 100644 --- a/elasticgraph-local/elasticgraph-local.gemspec +++ b/elasticgraph-local/elasticgraph-local.gemspec @@ -16,8 +16,9 @@ ElasticGraphGemspecHelper.define_elasticgraph_gem(gemspec_file: __FILE__, catego spec.add_dependency "elasticgraph-indexer", eg_version spec.add_dependency "elasticgraph-rack", eg_version spec.add_dependency "elasticgraph-schema_definition", eg_version - spec.add_dependency "rackup", "~> 2.1" + spec.add_dependency "rackup", "~> 2.2" spec.add_dependency "rake", "~> 13.2" + spec.add_dependency "webrick", "~> 1.9" spec.add_development_dependency "elasticgraph-elasticsearch", eg_version spec.add_development_dependency "elasticgraph-opensearch", eg_version