thrift 0.23.0 → 0.24.0

data/ext/memory_buffer.c

@@ -28,11 +28,15 @@ ID index_ivar_id;
 ID slice_method_id;
 
 int GARBAGE_BUFFER_SIZE;
+static VALUE transport_exception_class;
+static VALUE transport_negative_size;
+static ID new_method_id;
 
 #define GET_BUF(self) rb_ivar_get(self, buf_ivar_id)
 
 VALUE rb_thrift_memory_buffer_write(VALUE self, VALUE str);
 VALUE rb_thrift_memory_buffer_read(VALUE self, VALUE length_value);
+VALUE rb_thrift_memory_buffer_read_all(VALUE self, VALUE length_value);
 VALUE rb_thrift_memory_buffer_read_byte(VALUE self);
 VALUE rb_thrift_memory_buffer_read_into_buffer(VALUE self, VALUE buffer_value, VALUE size_value);
 
@@ -69,6 +73,33 @@ VALUE rb_thrift_memory_buffer_read(VALUE self, VALUE length_value) {
   return data;
 }
 
+VALUE rb_thrift_memory_buffer_read_all(VALUE self, VALUE length_value) {
+  int length = FIX2INT(length_value);
+
+  if (RB_UNLIKELY(length < 0)) {
+    rb_exc_raise(rb_funcall(transport_exception_class, new_method_id, 2, transport_negative_size, rb_str_new2("Negative size")));
+  }
+
+  VALUE index_value = rb_ivar_get(self, index_ivar_id);
+  int index = FIX2INT(index_value);
+  VALUE buf = GET_BUF(self);
+
+  if (RB_UNLIKELY(length > RSTRING_LEN(buf) - index)) {
+    rb_raise(rb_eEOFError, "Not enough bytes remain in memory buffer");
+  }
+
+  VALUE data = rb_str_subseq(buf, index, length);
+
+  index += length;
+  if (index >= GARBAGE_BUFFER_SIZE) {
+    rb_ivar_set(self, buf_ivar_id, rb_str_subseq(buf, index, RSTRING_LEN(buf) - index));
+    index = 0;
+  }
+  rb_ivar_set(self, index_ivar_id, INT2FIX(index));
+
+  return data;
+}
+
 VALUE rb_thrift_memory_buffer_read_byte(VALUE self) {
   VALUE index_value = rb_ivar_get(self, index_ivar_id);
   int index = FIX2INT(index_value);
@@ -122,6 +153,7 @@ void Init_memory_buffer(void) {
   VALUE thrift_memory_buffer_class = rb_const_get(thrift_module, rb_intern("MemoryBufferTransport"));
   rb_define_method(thrift_memory_buffer_class, "write", rb_thrift_memory_buffer_write, 1);
   rb_define_method(thrift_memory_buffer_class, "read", rb_thrift_memory_buffer_read, 1);
+  rb_define_method(thrift_memory_buffer_class, "read_all", rb_thrift_memory_buffer_read_all, 1);
   rb_define_method(thrift_memory_buffer_class, "read_byte", rb_thrift_memory_buffer_read_byte, 0);
   rb_define_method(thrift_memory_buffer_class, "read_into_buffer", rb_thrift_memory_buffer_read_into_buffer, 2);
 
@@ -129,6 +161,11 @@ void Init_memory_buffer(void) {
   index_ivar_id = rb_intern("@index");
 
   slice_method_id = rb_intern("slice");
+  new_method_id = rb_intern("new");
 
   GARBAGE_BUFFER_SIZE = FIX2INT(rb_const_get(thrift_memory_buffer_class, rb_intern("GARBAGE_BUFFER_SIZE")));
+  transport_exception_class = rb_const_get(thrift_module, rb_intern("TransportException"));
+  transport_negative_size = rb_const_get(transport_exception_class, rb_intern("NEGATIVE_SIZE"));
+  rb_global_variable(&transport_exception_class);
+  rb_global_variable(&transport_negative_size);
 }

data/ext/struct.c

@@ -35,8 +35,8 @@ static ID sorted_field_ids_method_id;
 #define IS_CONTAINER(ttype) ((ttype) == TTYPE_MAP || (ttype) == TTYPE_LIST || (ttype) == TTYPE_SET)
 #define STRUCT_FIELDS(obj) rb_const_get(CLASS_OF(obj), fields_const_id)
 
-static VALUE new_container_array(int size) {
-  if (size < 0) {
+static void validate_container_size(int size) {
+  if (RB_UNLIKELY(size < 0)) {
     rb_exc_raise(
       get_protocol_exception(
         INT2FIX(PROTOERR_NEGATIVE_SIZE),
@@ -44,7 +44,10 @@ static VALUE new_container_array(int size) {
       )
     );
   }
+}
 
+static VALUE new_container_array(int size) {
+  validate_container_size(size);
   return rb_ary_new2(size > 1024 ? 1024 : size);
 }
 
@@ -252,8 +255,6 @@ static void write_container(int ttype, VALUE field_info, VALUE value, VALUE prot
 
   if (ttype == TTYPE_MAP) {
     VALUE keys;
-    VALUE key;
-    VALUE val;
 
     Check_Type(value, T_HASH);
 
@@ -272,8 +273,8 @@ static void write_container(int ttype, VALUE field_info, VALUE value, VALUE prot
     default_write_map_begin(protocol, keytype_value, valuetype_value, INT2FIX(sz));
 
     for (i = 0; i < sz; i++) {
-      key = rb_ary_entry(keys, i);
-      val = rb_hash_aref(value, key);
+      VALUE key = rb_ary_entry(keys, i);
+      VALUE val = rb_hash_aref(value, key);
 
       if (IS_CONTAINER(keytype)) {
         write_container(keytype, key_info, key, protocol);
@@ -489,8 +490,6 @@ static VALUE read_anything(VALUE protocol, int ttype, VALUE field_info) {
       rb_thrift_struct_read(result, protocol);
     }
   } else if (ttype == TTYPE_MAP) {
-    int i;
-
     VALUE map_header = default_read_map_begin(protocol);
     int key_ttype = FIX2INT(rb_ary_entry(map_header, 0));
     int value_ttype = FIX2INT(rb_ary_entry(map_header, 1));
@@ -511,7 +510,7 @@ static VALUE read_anything(VALUE protocol, int ttype, VALUE field_info) {
       if (num_entries == 0 || (specified_key_type == key_ttype && specified_value_type == value_ttype)) {
         result = rb_hash_new();
 
-        for (i = 0; i < num_entries; ++i) {
+        for (int i = 0; i < num_entries; ++i) {
           VALUE key, val;
 
           key = read_anything(protocol, key_ttype, key_info);
@@ -528,8 +527,6 @@ static VALUE read_anything(VALUE protocol, int ttype, VALUE field_info) {
 
     default_read_map_end(protocol);
   } else if (ttype == TTYPE_LIST) {
-    int i;
-
     VALUE list_header = default_read_list_begin(protocol);
     int element_ttype = FIX2INT(rb_ary_entry(list_header, 0));
     int num_elements = FIX2INT(rb_ary_entry(list_header, 1));
@@ -542,20 +539,21 @@ static VALUE read_anything(VALUE protocol, int ttype, VALUE field_info) {
       if (specified_element_type == element_ttype) {
         result = new_container_array(num_elements);
 
-        for (i = 0; i < num_elements; ++i) {
+        for (int i = 0; i < num_elements; ++i) {
           rb_ary_push(result, read_anything(protocol, element_ttype, rb_hash_aref(field_info, element_sym)));
         }
       } else {
+        validate_container_size(num_elements);
         skip_list_or_set_contents(protocol, INT2FIX(element_ttype), num_elements);
       }
     } else {
+      validate_container_size(num_elements);
       skip_list_or_set_contents(protocol, INT2FIX(element_ttype), num_elements);
     }
 
     default_read_list_end(protocol);
   } else if (ttype == TTYPE_SET) {
     VALUE items;
-    int i;
 
     VALUE set_header = default_read_set_begin(protocol);
     int element_ttype = FIX2INT(rb_ary_entry(set_header, 0));
@@ -569,15 +567,17 @@ static VALUE read_anything(VALUE protocol, int ttype, VALUE field_info) {
       if (specified_element_type == element_ttype) {
         items = new_container_array(num_elements);
 
-        for (i = 0; i < num_elements; ++i) {
+        for (int i = 0; i < num_elements; ++i) {
           rb_ary_push(items, read_anything(protocol, element_ttype, rb_hash_aref(field_info, element_sym)));
         }
 
         result = rb_class_new_instance(1, &items, rb_cSet);
       } else {
+        validate_container_size(num_elements);
         skip_list_or_set_contents(protocol, INT2FIX(element_ttype), num_elements);
       }
     } else {
+      validate_container_size(num_elements);
       skip_list_or_set_contents(protocol, INT2FIX(element_ttype), num_elements);
     }
 

data/ext/thrift_native.c

@@ -121,6 +121,7 @@ int PROTOERR_BAD_VERSION;
 int PROTOERR_NOT_IMPLEMENTED;
 int PROTOERR_DEPTH_LIMIT;
 
+// cppcheck-suppress unusedFunction
 RUBY_FUNC_EXPORTED void Init_thrift_native(void) {
   // cached classes
   thrift_module = rb_const_get(rb_cObject, rb_intern("Thrift"));

data/lib/thrift/bytes.rb

@@ -1,4 +1,5 @@
 # encoding: ascii-8bit
+# frozen_string_literal: true
 #
 # Licensed to the Apache Software Foundation (ASF) under one
 # or more contributor license agreements. See the NOTICE file
@@ -36,13 +37,15 @@ module Thrift
     end
 
     # Forces the encoding of the buffer to BINARY. If the buffer
-    # passed is frozen, then it will be duplicated.
+    # passed is frozen and not already BINARY, then it will be duplicated.
     #
     # buffer - The String to force the encoding of.
     #
     # Returns the String passed with an encoding of BINARY; returned
     # String may be a duplicate.
     def self.force_binary_encoding(buffer)
+      return buffer if buffer.encoding == Encoding::BINARY
+
       buffer = buffer.dup if buffer.frozen?
       buffer.force_encoding(Encoding::BINARY)
     end

data/lib/thrift/client.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 #
 # Licensed to the Apache Software Foundation (ASF) under one
 # or more contributor license agreements. See the NOTICE file
@@ -61,11 +62,6 @@ module Thrift
       [fname, mtype, rseqid]
     end
 
-    def reply_seqid(rseqid)
-      expected_seqid = dequeue_pending_seqid
-      !expected_seqid.nil? && rseqid == expected_seqid
-    end
-
     def validate_message_begin(fname, mtype, rseqid, expected_name)
       expected_seqid = dequeue_pending_seqid
 
@@ -102,13 +98,6 @@ module Thrift
       result
     end
 
-    def handle_exception(mtype)
-      if mtype == MessageTypes::EXCEPTION
-        dequeue_pending_seqid
-        raise_application_exception
-      end
-    end
-
     private
 
     def next_seqid!

data/lib/thrift/exceptions.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 #
 # Licensed to the Apache Software Foundation (ASF) under one
 # or more contributor license agreements. See the NOTICE file

data/lib/thrift/multiplexed_processor.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 #
 # Licensed to the Apache Software Foundation (ASF) under one
 # or more contributor license agreements. See the NOTICE file
@@ -23,17 +24,26 @@ module Thrift
   class MultiplexedProcessor
     def initialize
       @actual_processors = {}
+      @default_processor = nil
     end
 
     def register_processor(service_name, processor)
       @actual_processors[service_name] = processor
     end
 
+    def register_default(processor)
+      @default_processor = processor
+    end
+
     def process(iprot, oprot)
       name, type, seqid = iprot.read_message_begin
       check_type(type)
-      check_separator(name)
-      service_name, method = name.split(':')
+      if name.count(':') < 1
+        check_default_processor(name)
+        return @default_processor.process(StoredMessageProtocol.new(iprot, [name, type, seqid]), oprot)
+      end
+
+      service_name, method = name.split(':', 2)
       processor(service_name).process(StoredMessageProtocol.new(iprot, [method, type, seqid]), oprot)
     end
 
@@ -53,8 +63,8 @@ module Thrift
       end
     end
 
-    def check_separator(name)
-      if name.count(':') < 1
+    def check_default_processor(name)
+      unless @default_processor
         raise Thrift::Exception.new("Service name not found in message name: #{name}. Did you forget to use a Thrift::Protocol::MultiplexedProtocol in your client?")
       end
     end

data/lib/thrift/processor.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 #
 # Licensed to the Apache Software Foundation (ASF) under one
 # or more contributor license agreements. See the NOTICE file

data/lib/thrift/protocol/base_protocol.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 #
 # Licensed to the Apache Software Foundation (ASF) under one
 # or more contributor license agreements. See the NOTICE file
@@ -341,7 +342,8 @@ module Thrift
       end
     end
 
-    def skip(type)
+    def skip(type, max_depth = 64)
+      raise ProtocolException.new(ProtocolException::DEPTH_LIMIT, 'Maximum skip depth exceeded') if max_depth <= 0
       case type
       when Types::BOOL
         read_bool
@@ -364,27 +366,30 @@ module Thrift
         while true
           name, type, id = read_field_begin
           break if type == Types::STOP
-          skip(type)
+          skip(type, max_depth - 1)
           read_field_end
         end
         read_struct_end
       when Types::MAP
         ktype, vtype, size = read_map_begin
+        validate_container_size(size)
         size.times do
-          skip(ktype)
-          skip(vtype)
+          skip(ktype, max_depth - 1)
+          skip(vtype, max_depth - 1)
         end
         read_map_end
       when Types::SET
         etype, size = read_set_begin
+        validate_container_size(size)
         size.times do
-          skip(etype)
+          skip(etype, max_depth - 1)
         end
         read_set_end
       when Types::LIST
         etype, size = read_list_begin
+        validate_container_size(size)
         size.times do
-          skip(etype)
+          skip(etype, max_depth - 1)
         end
         read_list_end
       else
@@ -392,6 +397,10 @@ module Thrift
       end
     end
 
+    def validate_container_size(size)
+      raise ProtocolException.new(ProtocolException::NEGATIVE_SIZE, 'Negative size') unless size >= 0
+    end
+
     def to_s
       "#{trans.to_s}"
     end

data/lib/thrift/protocol/binary_protocol.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 #
 # Licensed to the Apache Software Foundation (ASF) under one
 # or more contributor license agreements. See the NOTICE file
@@ -22,6 +23,14 @@ module Thrift
     VERSION_MASK = 0xffff0000
     VERSION_1 = 0x80010000
     TYPE_MASK = 0x000000ff
+    BYTE_MIN = -2**7
+    BYTE_MAX = 2**7 - 1
+    I16_MIN = -2**15
+    I16_MAX = 2**15 - 1
+    I32_MIN = -2**31
+    I32_MAX = 2**31 - 1
+    I64_MIN = -2**63
+    I64_MAX = 2**63 - 1
 
     attr_reader :strict_read, :strict_write
 
@@ -36,11 +45,10 @@ module Thrift
     end
 
     def write_message_begin(name, type, seqid)
-      # this is necessary because we added (needed) bounds checking to
-      # write_i32, and 0x80010000 is too big for that.
       if strict_write
-        write_i16(VERSION_1 >> 16)
-        write_i16(type)
+        raise ::TypeError, 'integer argument expected' unless type.is_a?(Integer)
+        raise RangeError if type < BYTE_MIN || type > BYTE_MAX
+        trans.write([VERSION_1 | type].pack('N'))
         write_string(name)
         write_i32(seqid)
       else
@@ -64,17 +72,17 @@ module Thrift
     def write_map_begin(ktype, vtype, size)
       write_byte(ktype)
       write_byte(vtype)
-      write_i32(size)
+      write_i32_size(size)
     end
 
     def write_list_begin(etype, size)
       write_byte(etype)
-      write_i32(size)
+      write_i32_size(size)
     end
 
     def write_set_begin(etype, size)
       write_byte(etype)
-      write_i32(size)
+      write_i32_size(size)
     end
 
     def write_bool(bool)
@@ -83,24 +91,29 @@ module Thrift
 
     def write_byte(byte)
       raise 'nil argument not allowed!' if byte.nil?
-      raise RangeError if byte < -2**31 || byte >= 2**32
+      raise ::TypeError, 'integer argument expected' unless byte.is_a?(Integer)
+      raise RangeError if byte < BYTE_MIN || byte > BYTE_MAX
       trans.write([byte].pack('c'))
     end
 
     def write_i16(i16)
       raise 'nil argument not allowed!' if i16.nil?
+      raise ::TypeError, 'integer argument expected' unless i16.is_a?(Integer)
+      raise RangeError if i16 < I16_MIN || i16 > I16_MAX
       trans.write([i16].pack('n'))
     end
 
     def write_i32(i32)
       raise 'nil argument not allowed!' if i32.nil?
-      raise RangeError if i32 < -2**31 || i32 >= 2**31
+      raise ::TypeError, 'integer argument expected' unless i32.is_a?(Integer)
+      raise RangeError if i32 < I32_MIN || i32 > I32_MAX
       trans.write([i32].pack('N'))
     end
 
     def write_i64(i64)
       raise 'nil argument not allowed!' if i64.nil?
-      raise RangeError if i64 < -2**63 || i64 >= 2**64
+      raise ::TypeError, 'integer argument expected' unless i64.is_a?(Integer)
+      raise RangeError if i64 < I64_MIN || i64 > I64_MAX
       hi = i64 >> 32
       lo = i64 & 0xffffffff
       trans.write([hi, lo].pack('N2'))
@@ -119,7 +132,7 @@ module Thrift
 
     def write_binary(buf)
       raise 'nil argument not allowed!' if buf.nil?
-      write_i32(buf.bytesize)
+      write_i32_size(buf.bytesize)
       trans.write(buf)
     end
 
@@ -165,18 +178,21 @@ module Thrift
       ktype = read_byte
       vtype = read_byte
       size = read_i32
+      raise ProtocolException.new(ProtocolException::NEGATIVE_SIZE, 'Negative size') unless size >= 0
       [ktype, vtype, size]
     end
 
     def read_list_begin
       etype = read_byte
       size = read_i32
+      raise ProtocolException.new(ProtocolException::NEGATIVE_SIZE, 'Negative size') unless size >= 0
       [etype, size]
     end
 
     def read_set_begin
       etype = read_byte
       size = read_i32
+      raise ProtocolException.new(ProtocolException::NEGATIVE_SIZE, 'Negative size') unless size >= 0
       [etype, size]
     end
 
@@ -236,7 +252,11 @@ module Thrift
 
     def read_binary
       size = read_i32
-      trans.read_all(size)
+      if size >= 0
+        trans.read_all(size)
+      else
+        raise ProtocolException.new(ProtocolException::NEGATIVE_SIZE, 'Negative size')
+      end
     end
 
     def read_uuid
@@ -246,6 +266,15 @@ module Thrift
     def to_s
       "binary(#{super.to_s})"
     end
+
+    private
+
+    def write_i32_size(size)
+      raise 'nil argument not allowed!' if size.nil?
+      raise ::TypeError, 'integer argument expected' unless size.is_a?(Integer)
+      raise RangeError if size < 0 || size > I32_MAX
+      trans.write([size].pack('N'))
+    end
   end
 
   class BinaryProtocolFactory < BaseProtocolFactory

data/lib/thrift/protocol/binary_protocol_accelerated.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 #
 # Licensed to the Apache Software Foundation (ASF) under one
 # or more contributor license agreements. See the NOTICE file

data/lib/thrift/protocol/compact_protocol.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 #
 # Licensed to the Apache Software Foundation (ASF) under one
 # or more contributor license agreements. See the NOTICE file
@@ -26,6 +27,8 @@ module Thrift
     TYPE_MASK = 0xE0
     TYPE_BITS = 0x07
     TYPE_SHIFT_AMOUNT = 5
+    MAX_VARINT32_BYTES = 5  # ceil(32/7); matches protobuf wire format
+    MAX_VARINT_BYTES = 10   # ceil(64/7); matches protobuf wire format
 
     TSTOP = [nil, Types::STOP, 0]
 
@@ -410,19 +413,27 @@ module Thrift
     end
 
     def read_varint32()
-      read_varint64()
+      shift = 0
+      result = 0
+      MAX_VARINT32_BYTES.times do
+        b = read_byte()
+        result |= (b & 0x7f) << shift
+        return result if (b & 0x80) != 0x80
+        shift += 7
+      end
+      raise ProtocolException.new(ProtocolException::INVALID_DATA, 'Variable-length int over 5 bytes.')
     end
 
     def read_varint64()
       shift = 0
       result = 0
-      while true
+      MAX_VARINT_BYTES.times do
         b = read_byte()
         result |= (b & 0x7f) << shift
-        break if (b & 0x80) != 0x80
+        return result if (b & 0x80) != 0x80
         shift += 7
       end
-      result
+      raise ProtocolException.new(ProtocolException::INVALID_DATA, 'Variable-length int over 10 bytes.')
     end
 
     def int_to_zig_zag(n)

data/lib/thrift/protocol/header_protocol.rb

@@ -1,4 +1,5 @@
 # encoding: ascii-8bit
+# frozen_string_literal: true
 #
 # Licensed to the Apache Software Foundation (ASF) under one
 # or more contributor license agreements. See the NOTICE file

data/lib/thrift/protocol/json_protocol.rb

@@ -1,4 +1,5 @@
 # encoding: UTF-8
+# frozen_string_literal: true
 #
 # Licensed to the Apache Software Foundation (ASF) under one
 # or more contributor license agreements. See the NOTICE file
@@ -500,10 +501,10 @@ module Thrift
     # characters above the BMP are encoded as two escape sequences (surrogate pairs),
     # which is not yet implemented
     def read_json_escape_char
-      str = @reader.read
-      str += @reader.read
-      str += @reader.read
-      str += @reader.read
+      str = +@reader.read
+      str << @reader.read
+      str << @reader.read
+      str << @reader.read
       str.hex.chr(Encoding::UTF_8)
     end
 
@@ -524,8 +525,7 @@ module Thrift
         @context.read(@reader)
       end
       read_json_syntax_char(@@kJSONStringDelimiter)
-      ch = ""
-      str = ""
+      str = +''
       while (true)
         ch = @reader.read
         if (ch == @@kJSONStringDelimiter)
@@ -543,7 +543,7 @@ module Thrift
             ch = escape_char_vals[pos]
           end
         end
-        str += ch
+        str << ch
       end
       return str
     end
@@ -564,14 +564,14 @@ module Thrift
     # Reads a sequence of characters, stopping at the first one that is not
     # a valid JSON numeric character.
     def read_json_numeric_chars
-      str = ""
+      str = String.new(encoding: Encoding::UTF_8)
       while (true)
         ch = @reader.peek
         if (!is_json_numeric(ch))
           break;
         end
         ch = @reader.read
-        str += ch
+        str << ch
       end
       return str
     end
@@ -712,6 +712,7 @@ module Thrift
       key_type = get_type_id_for_type_name(read_json_string)
       val_type = get_type_id_for_type_name(read_json_string)
       size = read_json_integer
+      validate_container_size(size)
       read_json_object_start
       [key_type, val_type, size]
     end
@@ -723,7 +724,10 @@ module Thrift
 
     def read_list_begin
       read_json_array_start
-      [get_type_id_for_type_name(read_json_string), read_json_integer]
+      type = get_type_id_for_type_name(read_json_string)
+      size = read_json_integer
+      validate_container_size(size)
+      [type, size]
     end
 
     def read_list_end
@@ -732,7 +736,10 @@ module Thrift
 
     def read_set_begin
       read_json_array_start
-      [get_type_id_for_type_name(read_json_string), read_json_integer]
+      type = get_type_id_for_type_name(read_json_string)
+      size = read_json_integer
+      validate_container_size(size)
+      [type, size]
     end
 
     def read_set_end

data/lib/thrift/protocol/multiplexed_protocol.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 #
 # Licensed to the Apache Software Foundation (ASF) under one
 # or more contributor license agreements. See the NOTICE file

data/lib/thrift/protocol/protocol_decorator.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 #
 # Licensed to the Apache Software Foundation (ASF) under one
 # or more contributor license agreements. See the NOTICE file

data/lib/thrift/serializer/deserializer.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 #
 # Licensed to the Apache Software Foundation (ASF) under one
 # or more contributor license agreements. See the NOTICE file

data/lib/thrift/serializer/serializer.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 #
 # Licensed to the Apache Software Foundation (ASF) under one
 # or more contributor license agreements. See the NOTICE file

data/lib/thrift/server/base_server.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 #
 # Licensed to the Apache Software Foundation (ASF) under one
 # or more contributor license agreements. See the NOTICE file