upfluence-thrift 2.7.9 → 2.7.12

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 687ae797f438d68fde1b522bd847e491496e75be163985198d6ca51c5bf529fd
-  data.tar.gz: cfcf375654d35ecfdaf044b291c8671f8cda5bd1992bbaa4d6036485d00b567b
+  metadata.gz: 9855b1966c6fd48cc31f981908b2fdcd8fd7e1b0637dd831bdcefaf9b7b779a8
+  data.tar.gz: 8aa5d8b3a438e5ab63e6d640cbb3f9d00c9c17073db85ca94dcf07471f8944a1
 SHA512:
-  metadata.gz: 55507a95b00d257938f2cae189ff5e7ae0efb2eba9ea3717b356c827b8add263fe4a3f6a1d08ffaa8e12a0fb6ee8bc019f70c153e24f1102b3d419f840988d40
-  data.tar.gz: c3873cc179fd0b58c31fa5dbfc4b19835197e10fd2cd6dfd2cb7da6af902c3d6050fb23408601cad3886f2e4944ce7305efbdbabfb3bcafc2b59fb3192b26cf9
+  metadata.gz: 53091d560bfdb7f7a4a8702028d47eed179823a92cdda0488b55c1ce047a676aa6e1111719ef1e7f52638e9872803205539084985253390a1778b7cc56dcdc1a
+  data.tar.gz: a4442f36ef65e752eaf1c291782a14c2da0d67772343c5c5c883e138a7213070ca50a8bbadeff635582f2be6a3f78b715e474453cce77aabde12cecfd52a866a

data/ext/struct.c

@@ -676,8 +676,9 @@ static VALUE rb_thrift_union_write(VALUE self, VALUE protocol) {
 
   VALUE ttype_value = rb_hash_aref(field_info, type_sym);
   int ttype = FIX2INT(ttype_value);
+  VALUE field_name = rb_hash_aref(field_info, name_sym);
 
-  default_write_field_begin(protocol, setfield, ttype_value, field_id);
+  default_write_field_begin(protocol, field_name, ttype_value, field_id);
 
   write_anything(ttype, setvalue, protocol, field_info);
 

data/lib/thrift/processor.rb

@@ -92,13 +92,14 @@ module Thrift
       end
       @middleware = Middleware.wrap(middlewares)
 
-      @processors = if self.class.const_defined? :METHODS
-                      self.class::METHODS.reduce({}) do |acc, (name, info)|
-                        acc.merge(name => build_processor(name, info))
-                      end
-                    else
-                      {}
-                    end
+      # Walk ancestors child first so inherited methods are exposed and overrides win.
+      @processors = self.class.ancestors.each_with_object({}) do |klass, acc|
+        next unless klass.const_defined?(:METHODS, false)
+
+        klass::METHODS.each do |name, info|
+          acc[name] ||= build_processor(name, info)
+        end
+      end
     end
 
     def read_args(iprot, args_class)

data/lib/thrift/union.rb

@@ -75,11 +75,11 @@ module Thrift
       field_info = struct_fields[fid]
       type = field_info[:type]
       if is_container? type
-        oprot.write_field_begin(@setfield, type, fid)
+        oprot.write_field_begin(field_info[:name], type, fid)
         write_container(oprot, @value, field_info)
         oprot.write_field_end
       else
-        oprot.write_field(@setfield, type, fid, @value)
+        oprot.write_field(field_info[:name], type, fid, @value)
       end
 
       oprot.write_field_stop

data/spec/json_protocol_spec.rb

@@ -33,6 +33,18 @@ describe 'JsonProtocol' do
       @trans.read(@trans.available).should == '{"greeting":"hello world"}'
     end
 
+    it 'should pretty print union' do
+      SpecNamespace::My_union.new(:integer32, 25).write(@prot)
+
+      @trans.read(@trans.available).should == '{"integer32":25}'
+    end
+
+    it 'should pretty print union with a container field' do
+      SpecNamespace::My_union.new(:my_map, { SpecNamespace::SomeEnum::ONE => [] }).write(@prot)
+
+      @trans.read(@trans.available).should == '{"my_map":{"0":[]}}'
+    end
+
     it 'shound not be able to read message' do
       @trans.write('{"greeting":"hello world"}')
       expect {@prot.read_message_begin}.to raise_error(Thrift::ProtocolException)

data/spec/processor_spec.rb

@@ -77,6 +77,20 @@ describe Thrift::Processor do
         result_klass: EmptyResult,
         args:         [],
         exceptions:   {},
+        void_result:  true,
+        oneway:       false
+      }
+    }.freeze
+  end
+
+  class ExtendedProcessor < CurrentProcessor
+    METHODS = {
+      'pong' => {
+        args_klass:   EmptyArgs,
+        result_klass: EmptyResult,
+        args:         [],
+        exceptions:   {},
+        void_result:  true,
         oneway:       false
       }
     }.freeze
@@ -179,6 +193,25 @@ describe Thrift::Processor do
     )
   end
 
+  %w[ping pong].each do |name|
+    it "processes #{name} with a processor extending another service" do
+      handler = double('handler', ping: nil, pong: nil)
+      processor = ExtendedProcessor.new(handler)
+      oprot, trans = output_protocol
+
+      expect(processor.process(request_protocol(name), oprot)).to eq(true)
+      expect(oprot.read_message_begin).to eq(
+        [name, Thrift::MessageTypes::REPLY, 17]
+      )
+
+      EmptyResult.new.read(oprot)
+      oprot.read_message_end
+
+      expect(trans.available).to eq(0)
+      expect(handler).to have_received(name)
+    end
+  end
+
   it 'writes internal errors raised by a current processor' do
     handler = double('handler')
     allow(handler).to receive(:ping).and_raise(StandardError, 'boom')

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: upfluence-thrift
 version: !ruby/object:Gem::Version
-  version: 2.7.9
+  version: 2.7.12
 platform: ruby
 authors:
 - Thrift Developers
 bindir: bin
 cert_chain: []
-date: 2026-09-01 00:00:00.000000000 Z
+date: 2026-09-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec