curb 1.3.1 → 1.3.3

data/tests/tc_curl_easy_resolve.rb

@@ -1,7 +1,10 @@
 require File.expand_path(File.join(File.dirname(__FILE__), 'helper'))
 
 class TestCurbCurlEasyResolve < Test::Unit::TestCase
+  include TestServerMethods
+
   def setup
+    server_setup
     @easy = Curl::Easy.new
   end
 
@@ -13,4 +16,33 @@ class TestCurbCurlEasyResolve < Test::Unit::TestCase
   def test_empty_resolve
     assert_equal @easy.resolve, nil
   end
+
+  def test_setopt_resolve_persists_across_performs
+    host = "curb-setopt-resolve.invalid"
+    mapping = "#{host}:#{TestServlet.port}:127.0.0.1"
+
+    @easy.url = "http://#{host}:#{TestServlet.port}#{TestServlet.path}"
+    @easy.dns_cache_timeout = 0
+    @easy.set(:resolve, [mapping])
+
+    @easy.perform
+    assert_match(/GET/, @easy.body_str)
+
+    @easy.perform
+    assert_match(/GET/, @easy.body_str)
+  end
+
+  def test_resolve_entries_can_be_objects_that_convert_to_string
+    host = "curb-resolve-object.invalid"
+    mapping = "#{host}:#{TestServlet.port}:127.0.0.1"
+    entry = Object.new
+    entry.define_singleton_method(:to_s) { mapping }
+
+    @easy.url = "http://#{host}:#{TestServlet.port}#{TestServlet.path}"
+    @easy.dns_cache_timeout = 0
+    @easy.resolve = [entry]
+
+    @easy.perform
+    assert_match(/GET/, @easy.body_str)
+  end
 end

data/tests/tc_curl_multi.rb

@@ -133,6 +133,62 @@ class TestCurbCurlMulti < Test::Unit::TestCase
     Curl::Multi.autoclose = false
   end
 
+  def test_close_inside_completion_callback_is_rejected_without_invalidating_multi
+    multi = Curl::Multi.new
+    easy = Curl::Easy.new(TestServlet.url)
+
+    easy.on_complete { multi.close }
+    multi.add(easy)
+
+    error = assert_raise(Curl::Err::AbortedByCallbackError) { multi.perform }
+    assert_match(/Cannot close an active Curl::Multi handle/, error.message)
+    assert multi.idle?
+    assert_equal({}, multi.requests)
+
+    easy.on_complete { |_curl| }
+    multi.add(easy)
+    multi.perform
+
+    assert_equal "GET", easy.body_str
+  ensure
+    multi.close if defined?(multi) && multi
+  end
+
+  def test_close_inside_perform_block_is_rejected_without_invalidating_multi
+    multi = Curl::Multi.new
+    easy = Curl::Easy.new(TestServlet.url)
+    multi.add(easy)
+
+    error = assert_raise(RuntimeError) do
+      multi.perform { multi.close }
+    end
+    assert_match(/Cannot close an active Curl::Multi handle/, error.message)
+
+    assert_nothing_raised { multi.close }
+    multi = nil
+  ensure
+    multi.close if defined?(multi) && multi
+  end
+
+  def test_active_easy_cannot_be_added_to_another_multi
+    first_multi = Curl::Multi.new
+    second_multi = Curl::Multi.new
+    easy = Curl::Easy.new(TestServlet.url)
+
+    first_multi.add(easy)
+
+    error = assert_raise(RuntimeError) { second_multi.add(easy) }
+    assert_match(/active Curl::Easy/, error.message)
+    assert_same first_multi, easy.multi
+    assert_equal easy, first_multi.requests[easy.object_id]
+
+    assert_nothing_raised { first_multi.close }
+    first_multi = nil
+  ensure
+    first_multi.close if defined?(first_multi) && first_multi
+    second_multi.close if defined?(second_multi) && second_multi
+  end
+
   def test_close_makes_multi_unusable
     multi = Curl::Multi.new
     multi.close

data/tests/tc_curl_native_coverage.rb

@@ -90,6 +90,21 @@ class TestCurbCurlNativeCoverage < Test::Unit::TestCase
     easy.close if defined?(easy) && easy
   end
 
+  def test_clone_rebinds_upload_callbacks_to_clone_state
+    easy = Curl::Easy.new(TestServlet.url)
+    easy.put_data = 'clone-data'
+
+    clone = easy.clone
+    easy.put_data = 'other-data'
+
+    clone.perform
+
+    assert_equal "PUT\nclone-data", clone.body_str
+  ensure
+    clone.close if defined?(clone) && clone
+    easy.close if defined?(easy) && easy
+  end
+
   def test_native_accessors_round_trip
     easy = Curl::Easy.new(TestServlet.url)
 

data/tests/tc_curl_postfield.rb

@@ -136,11 +136,26 @@ class TestCurbCurlPostfield < Test::Unit::TestCase
     assert_equal "foo=FOOBAR", pf.to_s
   end
 
+  def test_content_proc_survives_gc
+    pf = postfield_with_only_native_proc_reference
+
+    10.times do
+      GC.start(full_mark: true, immediate_sweep: true)
+      GC.compact if GC.respond_to?(:compact)
+    end
+
+    assert_equal "foo=FOOBAR", pf.to_s
+  end
+
   def test_to_s_04
     pf = Curl::PostField.file('foo.file', 'bar.file')
     assert_nothing_raised { pf.to_s }
     #assert_raise(Curl::Err::InvalidPostFieldError) { pf.to_s }
   end
+
+  def postfield_with_only_native_proc_reference
+    Curl::PostField.content('foo') { |field| field.name.upcase + "BAR" }
+  end
 end
 
 class TestCurbCurlPostfieldNativeCoverage < Test::Unit::TestCase

data/tests/tc_ftp_options.rb

@@ -22,5 +22,18 @@ class TestCurbFtpOptions < Test::Unit::TestCase
     assert_kind_of(Array, c.ftp_commands)
     assert_equal ["PWD", "CWD /"], c.ftp_commands
   end
-end
 
+  def test_ftp_command_entries_can_be_objects_that_convert_to_string
+    command = Object.new
+    command.define_singleton_method(:to_s) { "PWD" }
+
+    c = Curl::Easy.new($TEST_URL)
+    c.ftp_commands = [command]
+    m = Curl::Multi.new
+
+    assert_nothing_raised { m.add(c) }
+  ensure
+    m.remove(c) if defined?(m) && m && m.requests[c.object_id]
+    m.close if defined?(m) && m
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: curb
 version: !ruby/object:Gem::Version
-  version: 1.3.1
+  version: 1.3.3
 platform: ruby
 authors:
 - Ross Bamford
 - Todd A. Fisher
 bindir: bin
 cert_chain: []
-date: 2026-04-05 00:00:00.000000000 Z
+date: 2026-05-11 00:00:00.000000000 Z
 dependencies: []
 description: Curb (probably CUrl-RuBy or something) provides Ruby-language bindings
   for the libcurl(3), a fully-featured client-side URL transfer library. cURL and