xapian_db 1.3.8 → 1.3.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a9a56c8dcca3d9af676803407a5f0ba9ca07d72a33c877909f9e5b7bc0ab884c
-  data.tar.gz: 19f87585356eb95bbe54c4e22d10b770b953c09c74856ffe10441fcf053c530d
+  metadata.gz: a219ad5e9648568bdf084f50f5639133fb493a5dd6ec44c59e2adb03102c4bdc
+  data.tar.gz: 472fa6a0894c26c46991d61cd891202aa444f92b661b2f3172856e050e6f30d5
 SHA512:
-  metadata.gz: 107aa90d69efc2dccd06f104a66fd9b0a252491c1110ebbc879e3998c74fd7619b1c41c523af7f8017611287ad339290f5a0aec72b07825687873af9f6780ae4
-  data.tar.gz: 3052ca4107227f97f48bbbc023478279e2f06c952b604d77269bb52ae3add718675940ce044a99b616037e801144ed09c474ad53ed7344853046e8200ce7d2aa
+  metadata.gz: 8f4084aed2f0fce0edc901f695d076d740a61b72ea8e2ccd9b7315af3b7b389478da74d176e2bebf019d8d7f57632595cdc9a436df875c9f5ad16541f49ae96f
+  data.tar.gz: 3f7463fab1bfdec7ff5c47306243c6c48ed10abfb0fbda970246bbc08289431e14c625ef6b98b53013182ac2ac5c9b07565739ec801f1e3add5f8720cf7f661a

data/CHANGELOG.md

@@ -1,3 +1,9 @@
+## 1.3.9 (October 26th, 2022)
+
+Fixes:
+
+- `XapianDb.execute_block` now uses `Thread.current[:xapian_db_block_writer]` to store its block-local writer as to avoid leaking into another Thread
+
 ## 1.3.8 (July 19th, 2021)
 
 Fixes:

data/lib/xapian_db.rb

@@ -106,21 +106,21 @@ module XapianDb
   # Update an object in the index
   # @param [Object] obj An instance of a class with a blueprint configuration
   def self.index(obj, commit=true, changed_attrs: [])
-    writer = @block_writer || XapianDb::Config.writer
+    writer = Thread.current[:xapian_db_block_writer] || XapianDb::Config.writer
     writer.index obj, commit, changed_attrs: changed_attrs
   end
 
   # Remove a document from the index
   # @param [String] xapian_id The document id
   def self.delete_doc_with(xapian_id, commit=true)
-    writer = @block_writer || XapianDb::Config.writer
+    writer = Thread.current[:xapian_db_block_writer] || XapianDb::Config.writer
     writer.delete_doc_with xapian_id, commit
   end
 
   # Update or delete a xapian document belonging to an object depending on the ignore_if logic(if present)
   # @param [Object] object An instance of a class with a blueprint configuration
   def self.reindex(object, commit=true, changed_attrs: [])
-    writer = @block_writer || XapianDb::Config.writer
+    writer = Thread.current[:xapian_db_block_writer] || XapianDb::Config.writer
     blueprint = XapianDb::DocumentBlueprint.blueprint_for object.class.name
     if blueprint.should_index?(object)
       writer.index object, commit, changed_attrs: changed_attrs
@@ -174,7 +174,7 @@ module XapianDb
   # @option opts [Object] :writer An index writer
   # @option opts [String] :error_message the error message to log if an error occurs
   def self.execute_block(opts, &block)
-    @block_writer = opts[:writer]
+    Thread.current[:xapian_db_block_writer] = opts[:writer]
     begin
       block.call
     rescue Exception => ex
@@ -188,7 +188,7 @@ module XapianDb
       raise
     ensure
       # release the block writer
-      @block_writer = nil
+      Thread.current[:xapian_db_block_writer] = nil
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: xapian_db
 version: !ruby/object:Gem::Version
-  version: 1.3.8
+  version: 1.3.9
 platform: ruby
 authors:
 - Gernot Kogler
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-07-19 00:00:00.000000000 Z
+date: 2022-10-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: daemons
@@ -260,8 +260,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: 1.3.6
 requirements: []
-rubyforge_project:
-rubygems_version: 2.7.6
+rubygems_version: 3.3.14
 signing_key:
 specification_version: 4
 summary: Ruby library to use a Xapian db as a key/value store with high performance