solr_wrapper 4.0.2 → 4.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d0ec6a2ca3c8e571ae16e5ded77d9786ce333094a0fc8773d61b46d2bf1ba211
-  data.tar.gz: 3636273764eb165f5dc132bb2a558fd13b974c91f0f6e3a6d1cfaa318fb1b327
+  metadata.gz: 751cc40ca1d014568d2ec3a7b6f388a34ddf64667e22b5b7aec00155a99762d8
+  data.tar.gz: 3204adf72ccae744917b3fdb53597be80a08a26cab38702ab70cb1c4902e255a
 SHA512:
-  metadata.gz: da1cfd3a2b98fc21125970c6357c2763394e6620f9b22df2ee1d08594d0948c0cb99f0051f4c79c76cd4f89c170db00b078f80687163a119fa6a5882a3c56c52
-  data.tar.gz: 5ebff3d507a4ff72b3fc711756c2f9bfb1e443c617b4a55d12da8f64ecbe655da417c5b3063f81dead29cf94acfc0d3baa1cb439af5936933c5b73e68beafe61
+  metadata.gz: 8c9bfd8aa35f953b95d4e2c0f570f40509cf27ec052e5444c182ebbd338bbe865ff44be9d015d9c7dfbbd6872f56baf70d89f126034bf6e2e3035caeadd6d532
+  data.tar.gz: f2aa2cb38e686787991488c8fa693a3693920151ac6497ecef497d88b191aacc682399bd4026457aeb59a9878434c7e843154af1f159bb1ff4af47664fca38bf

data/.github/workflows/ci.yml

@@ -12,11 +12,19 @@ jobs:
     strategy:
       matrix:
         ruby:
-          - 2.7
-          - 3.0
-          - jruby-9.2.20.0
+          - '3.2.5'
+          - '3.3.4'
+          - jruby-9.4.8.0
+        solr_version: ['']
+        include:
+          - ruby: '3.3.4'
+            solr_version: '8.11.4'
+          - ruby: '3.3.4'
+            solr_version: '9.6.1'
+    env:
+      SOLR_WRAPPER_SOLR_VERSION: ${{ matrix.solr_version }}
     steps:
-    - uses: actions/checkout@v2
+    - uses: actions/checkout@v3
     - name: Set up Ruby
       uses: ruby/setup-ruby@v1
       with:

data/lib/solr_wrapper/checksum_validator.rb

@@ -13,13 +13,16 @@ module SolrWrapper
 
     def validate?(file)
       return true if config.validate == false
-      Digest.const_get(algorithm.upcase).file(file).hexdigest == expected_sum(algorithm)
+
+      actual_sum(file) == expected_sum
     end
 
     def validate!(file)
-      unless validate? file
-        raise "Checksum mismatch" unless config.ignore_checksum
-      end
+      return if validate?(file)
+
+      return if config.ignore_checksum || defined?(JRUBY_VERSION)
+
+      raise "Checksum mismatch: #{file} (expected(#{expected_sum}) != actual(#{actual_sum(file)})"
     end
 
     private
@@ -36,8 +39,12 @@ module SolrWrapper
         File.join(config.download_dir, File.basename(checksumurl(suffix)))
       end
 
-      def expected_sum(alg)
-        config.checksum || read_file(alg)
+      def actual_sum(file)
+        Digest.const_get(algorithm.upcase).file(file).hexdigest
+      end
+
+      def expected_sum
+        config.checksum || read_file(algorithm)
       end
 
       def read_file(alg)

data/lib/solr_wrapper/client.rb

@@ -1,4 +1,4 @@
-require 'http'
+require 'faraday'
 require 'json'
 
 module SolrWrapper
@@ -18,7 +18,7 @@ module SolrWrapper
     private
 
     def collection?(name)
-      response = HTTP.get("#{url}admin/collections?action=LIST&wt=json")
+      response = Faraday.get("#{url}admin/collections?action=LIST&wt=json")
       data = JSON.parse(response.body)
       return if data['error'] && data['error']['msg'] == 'Solr instance is not running in SolrCloud mode.'
 
@@ -26,7 +26,7 @@ module SolrWrapper
     end
 
     def core?(name)
-      response = HTTP.get("#{url}admin/cores?action=STATUS&wt=json&core=#{name}")
+      response = Faraday.get("#{url}admin/cores?action=STATUS&wt=json&core=#{name}")
       !JSON.parse(response.body)['status'][name].empty?
     end
   end

data/lib/solr_wrapper/configuration.rb

@@ -1,4 +1,5 @@
-require 'http'
+require 'faraday'
+require 'faraday/follow_redirects'
 
 module SolrWrapper
   class Configuration
@@ -9,6 +10,7 @@ module SolrWrapper
       @verbose = options[:verbose]
 
       @options = load_configs(Array(options[:config])).merge(options)
+      @options[:env] ||= ENV
     end
 
     def solr_xml
@@ -84,7 +86,12 @@ module SolrWrapper
 
     def version
       @version ||= begin
-        config_version = options.fetch(:version, SolrWrapper.default_instance_options[:version])
+        config_version = if env_options[:version].nil? || env_options[:version].empty?
+          options.fetch(:version, SolrWrapper.default_instance_options[:version])
+        else
+          env_options[:version]
+        end
+
         if config_version == 'latest'
           fetch_latest_version
         else
@@ -110,18 +117,23 @@ module SolrWrapper
         options[:mirror_url] + mirror_artifact_path
       else
         begin
-          json = HTTP.follow.get(closest_mirror_url).body
+          client = Faraday.new(closest_mirror_url) do |faraday|
+            faraday.use Faraday::FollowRedirects::Middleware
+            faraday.adapter Faraday.default_adapter
+          end
+
+          json = client.get.body
           doc = JSON.parse(json)
           url = doc['preferred'] + doc['path_info']
 
-          response = HTTP.head(url)
+          response = Faraday.head(url)
 
-          if response.status.success?
+          if response.success?
             url
           else
             archive_download_url
           end
-        rescue Errno::ECONNRESET, SocketError, HTTP::Error
+        rescue Errno::ECONNRESET, SocketError, Faraday::Error
           archive_download_url
         end
       end
@@ -210,8 +222,24 @@ module SolrWrapper
       end
 
       def fetch_latest_version
-        response = HTTP.follow.get(options.fetch(:latest_version_url, 'https://solr.apache.org/downloads.html'))
+        url = options.fetch(:latest_version_url, 'https://solr.apache.org/downloads.html')
+
+        client = Faraday.new(url) do |faraday|
+          faraday.use Faraday::FollowRedirects::Middleware
+          faraday.adapter Faraday.default_adapter
+        end
+
+        response = client.get
         response.body.to_s[/Solr \d+\.\d+\.\d+/][/\d+\.\d+\.\d+/]
       end
+
+      def env_options
+        @env_options ||= begin
+          env = options.fetch(:env, {})
+          {
+            version: env['SOLR_WRAPPER_SOLR_VERSION']
+          }
+        end
+      end
   end
 end

data/lib/solr_wrapper/downloader.rb

@@ -1,23 +1,35 @@
 require 'ruby-progressbar'
-require 'http'
+require 'faraday'
+require 'faraday/follow_redirects'
 
 module SolrWrapper
   class Downloader
     def self.fetch_with_progressbar(url, output)
       pbar = SafeProgressBar.new(title: File.basename(url), total: nil, format: '%t: |%B| %p%% (%e )')
 
-      response = HTTP.follow.get(url)
-      pbar.total = response.headers['content-length'].to_i
+      client = Faraday.new(url) do |faraday|
+        faraday.use Faraday::FollowRedirects::Middleware
+        faraday.adapter Faraday.default_adapter
+      end
 
       File.open(output, 'wb') do |f|
-        response.body.each do |chunk|
-          f.write(chunk)
-          pbar.progress += chunk.length
-        end
+        client.get do |req|
+          req.options.on_data = Proc.new do |chunk, overall_received_bytes, env|
+            if env
+              pbar.total = env.response_headers['content-length'].to_i
+              pbar.progress = overall_received_bytes
+            else
+              pbar.increment
+            end
 
-        nil
+            pbar.progress = overall_received_bytes
+            f.write(chunk)
+          end
+        end
       end
-    rescue HTTP::Error => e
+
+      true
+    rescue Faraday::Error => e
       raise SolrWrapperError, "Unable to download solr from #{url}\n#{e}"
     end
 

data/lib/solr_wrapper/instance.rb

@@ -144,23 +144,52 @@ module SolrWrapper
     # @option options [String] :name
     # @option options [String] :dir
     def create(options = {})
-      options[:name] ||= SecureRandom.hex
+      tmpdir = nil
 
-      create_options = { p: port }
-      create_options[:c] = options[:name] if options[:name]
-      create_options[:n] = options[:config_name] if options[:config_name]
-      create_options[:d] = options[:dir] if options[:dir]
+      begin
+        options[:name] ||= SecureRandom.hex
 
-      Retriable.retriable do
-        raise "Not started yet" unless started?
-      end
+        create_options = {}
+        if version >= '9.7'
+          create_options[:url] = config.base_url
+        else
+          create_options[:p] = port
+        end
+        create_options[:c] = options[:name] if options[:name]
+        create_options[:n] = options[:config_name] if options[:config_name]
+
+        if options[:dir]
+          # Solr 9.7 required that the dir argument contain a `conf` directory that
+          # contains the actual configuration files.
+          if version >= '9.7' && !File.exist?(File.join(options[:dir], 'conf'))
+
+            if options[:dir].match?(/conf\/$/)
+              create_options[:d] = File.expand_path(options[:dir], '..')
+            else
+              tmpdir = Dir.mktmpdir
+
+              FileUtils.mkdir("#{tmpdir}/conf")
+              FileUtils.cp_r("#{options[:dir]}/.", "#{tmpdir}/conf")
+              create_options[:d] = tmpdir
+            end
+          else
+            create_options[:d] = options[:dir]
+          end
+        end
+
+        Retriable.retriable do
+          raise "Not started yet" unless started?
+        end
 
-      # short-circuit if we're using persisted data with an existing core/collection
-      return if options[:persist] && create_options[:c] && client.exists?(create_options[:c])
+        # short-circuit if we're using persisted data with an existing core/collection
+        return if options[:persist] && create_options[:c] && client.exists?(create_options[:c])
 
-      exec("create", create_options)
+        exec("create", create_options)
 
-      options[:name]
+        options[:name]
+      ensure
+        FileUtils.remove_entry tmpdir if tmpdir
+      end
     end
 
     ##
@@ -172,11 +201,11 @@ module SolrWrapper
       options[:name] ||= SecureRandom.hex
       options[:zkhost] ||= zkhost
 
-      upconfig_options = { upconfig: true, n: options[:name] }
+      upconfig_options = { n: options[:name] }
       upconfig_options[:d] = options[:dir] if options[:dir]
       upconfig_options[:z] = options[:zkhost] if options[:zkhost]
 
-      exec 'zk', upconfig_options
+      exec ['zk', 'upconfig'], upconfig_options
 
       options[:name]
     end
@@ -190,11 +219,11 @@ module SolrWrapper
       options[:name] ||= SecureRandom.hex
       options[:zkhost] ||= zkhost
 
-      downconfig_options = { downconfig: true, n: options[:name] }
+      downconfig_options = { n: options[:name] }
       downconfig_options[:d] = options[:dir] if options[:dir]
       downconfig_options[:z] = options[:zkhost] if options[:zkhost]
 
-      exec 'zk', downconfig_options
+      exec ['zk', 'downconfig'], downconfig_options
 
       options[:name]
     end
@@ -203,7 +232,15 @@ module SolrWrapper
     # Create a new collection in solr
     # @param [String] name collection name
     def delete(name, _options = {})
-      exec("delete", c: name, p: port)
+      delete_options = { c: name }
+
+      if version >= '9.7'
+        delete_options[:url] = config.base_url
+      else
+        delete_options[:p] = port
+      end
+
+      exec("delete", delete_options)
     end
 
     ##

data/lib/solr_wrapper/runner.rb

@@ -17,7 +17,7 @@ module SolrWrapper
     private
 
     def argument_list
-      [config.solr_binary, cmd] + config.solr_options.merge(options).map do |k, v|
+      [config.solr_binary] + Array(cmd) + config.solr_options.merge(options).map do |k, v|
         case v
         when true
           "-#{k}"

data/lib/solr_wrapper/settings.rb

@@ -42,10 +42,14 @@ module SolrWrapper
       @zookeeper_port ||= "#{port.to_i + 1000}"
     end
 
+    def base_url
+      "http://#{host}:#{port}/"
+    end
+
     ##
     # Get a (likely) URL to the solr instance
     def url
-      "http://#{host}:#{port}/solr/"
+      "#{base_url}solr/"
     end
 
     def instance_dir

data/lib/solr_wrapper/version.rb

@@ -1,3 +1,3 @@
 module SolrWrapper
-  VERSION = '4.0.2'
+  VERSION = '4.1.1'
 end

data/solr_wrapper.gemspec

@@ -18,16 +18,17 @@ Gem::Specification.new do |spec|
   spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
   spec.require_paths = ["lib"]
 
-  spec.add_dependency "http"
+  spec.add_dependency "faraday", "~> 2.5"
+  spec.add_dependency "faraday-follow_redirects"
   spec.add_dependency "minitar"
   spec.add_dependency "ruby-progressbar"
   spec.add_dependency "retriable"
+  spec.add_dependency "ostruct"
 
   spec.add_development_dependency "bundler", ">= 1.7", "< 3"
-  spec.add_development_dependency "rake", "~> 10.0", "< 13"
+  spec.add_development_dependency "rake", ">= 12.2", "< 14"
 
   spec.add_development_dependency "rspec"
   spec.add_development_dependency "simple_solr_client", "= 0.2.0" # 0.2.1 removed support for schema retrieval
-  spec.add_development_dependency "coveralls"
   spec.add_development_dependency "webmock"
 end

data/spec/fixtures/basic_configs/solrconfig.xml

@@ -240,7 +240,7 @@
            autowarmCount - the number of entries to prepopulate from
                and old cache.  
       -->
-    <filterCache class="solr.FastLRUCache"
+    <filterCache class="solr.CaffeineCache"
                  size="512"
                  initialSize="512"
                  autowarmCount="0"/>
@@ -250,7 +250,7 @@
          Caches results of searches - ordered lists of document ids
          (DocList) based on a query, a sort, and the range of documents requested.  
       -->
-    <queryResultCache class="solr.LRUCache"
+    <queryResultCache class="solr.CaffeineCache"
                      size="512"
                      initialSize="512"
                      autowarmCount="0"/>
@@ -261,14 +261,14 @@
          document).  Since Lucene internal document ids are transient,
          this cache will not be autowarmed.  
       -->
-    <documentCache class="solr.LRUCache"
+    <documentCache class="solr.CaffeineCache"
                    size="512"
                    initialSize="512"
                    autowarmCount="0"/>
     
     <!-- custom cache currently used by block join --> 
     <cache name="perSegFilter"
-      class="solr.search.LRUCache"
+      class="solr.CaffeineCache"
       size="10"
       initialSize="0"
       autowarmCount="10"

data/spec/lib/solr_wrapper/configuration_spec.rb

@@ -1,7 +1,8 @@
 require 'spec_helper'
 
 describe SolrWrapper::Configuration do
-  let(:config) { described_class.new options }
+  let(:config) { described_class.new default_options.merge(options) }
+  let(:default_options) { { env: {} } }
 
   describe "#port" do
     subject { config.port }

data/spec/lib/solr_wrapper/instance_spec.rb

@@ -135,7 +135,7 @@ describe SolrWrapper::Instance do
       let(:cmd) { 'healthcheck' }
       let(:options) { { z: 'localhost:5098' } }
       it 'raises an error with the output from the shell command' do
-        expect { subject }.to raise_error(RuntimeError, /Failed to execute solr healthcheck: collection parameter is required!/)
+        expect { subject }.to raise_error(RuntimeError, Regexp.union(/Failed to execute solr healthcheck: collection parameter is required!/, /Failed to parse command-line arguments due to: Missing required option: c/))
       end
     end
   end

data/spec/spec_helper.rb

@@ -1,6 +1,3 @@
-require 'coveralls'
-Coveralls.wear!
-
 require 'solr_wrapper'
 require 'simple_solr_client'
 

metadata

@@ -1,17 +1,31 @@
 --- !ruby/object:Gem::Specification
 name: solr_wrapper
 version: !ruby/object:Gem::Version
-  version: 4.0.2
+  version: 4.1.1
 platform: ruby
 authors:
 - Chris Beer
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2022-05-23 00:00:00.000000000 Z
+date: 2025-03-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: http
+  name: faraday
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.5'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.5'
+- !ruby/object:Gem::Dependency
+  name: faraday-follow_redirects
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -66,6 +80,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: ostruct
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -90,22 +118,22 @@ dependencies:
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '12.2'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '13'
+        version: '14'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '12.2'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '13'
+        version: '14'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -134,20 +162,6 @@ dependencies:
     - - '='
       - !ruby/object:Gem::Version
         version: 0.2.0
-- !ruby/object:Gem::Dependency
-  name: coveralls
-  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: webmock
   requirement: !ruby/object:Gem::Requirement
@@ -230,7 +244,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.32
+rubygems_version: 3.5.9
 signing_key:
 specification_version: 4
 summary: Solr service wrapper