protobuf 2.5.0 → 2.5.1

data/README.md

@@ -14,6 +14,8 @@ So let's dive in and see how to work with all three.
 
 ## 1. Generating ruby classes from `.proto` files
 
+_The `protobuf` package is required for compilation. Mac: `brew install protobuf`, Ubuntu: `sudo apt-get install -y protobuf`_
+
 Protocol Buffers are great because they allow you to clearly define data storage or data transfer packets. Google officially supports Java, C++, and Python for compilation and usage. Let's make it ruby aware!
 
 Let's say you have a `defs.proto` file that defines a User message.
@@ -24,9 +26,11 @@ message User {
   required string first_name = 1;
   required string last_name = 2;
 }
+```
 
 Now let's compile that definition to ruby:
 
+```
 $ rprotoc defs.proto --ruby_out ./lib
 ```
 

data/lib/protobuf.rb

@@ -14,6 +14,19 @@ module Protobuf
 
 	module_function
 
+  # Client Host
+  #
+  # Default: `hostname` of the system
+  #
+  # The name or address of the host to use during client RPC calls.
+  def self.client_host
+    @_client_host ||= `hostname`.chomp
+  end
+
+  def self.client_host=(host)
+    @_client_host = host
+  end
+
   # Connector Type
   #
   # Default: socket

data/lib/protobuf/rpc/connectors/base.rb

@@ -9,14 +9,15 @@ module Protobuf
   module Rpc
     module Connectors
       DEFAULT_OPTIONS = {
-        :service        => nil,           # Service to invoke
-        :method         => nil,           # Service method to call
-        :host           => '127.0.0.1',   # A default host (usually overridden)
-        :port           => '9399',        # A default port (usually overridden)
+        :service        => nil,           # Fully-qualified Service class
+        :method         => nil,           # Service method to invoke
+        :host           => '127.0.0.1',   # The hostname or address of the service (usually overridden or pre-configured)
+        :port           => '9399',        # The port of the service (usually overridden or pre-configured)
         :request        => nil,           # The request object sent by the client
         :request_type   => nil,           # The request type expected by the client
         :response_type  => nil,           # The response type expected by the client
-        :timeout        => 30             # The default timeout for the request, also handled by client.rb
+        :timeout        => 30,            # The default timeout for the request, also handled by client.rb
+        :client_host    => nil            # The hostname or address of this client
       }
 
       class Base

data/lib/protobuf/rpc/connectors/common.rb

@@ -11,6 +11,10 @@ module Protobuf
           end
         end
 
+        def request_caller
+          @options[:client_host] || ::Protobuf.client_host
+        end
+
         def complete
           @stats.stop
           log_info { @stats.to_s }
@@ -101,21 +105,21 @@ module Protobuf
           fail(:RPC_ERROR, "Connection error: #{e.message}")
         end
 
-        def rpc_request_data
-          validate_request_type
+        def request_bytes
+          validate_request_type!
+          fields = { :service_name => @options[:service].name,
+                     :method_name => @options[:method].to_s,
+                     :request_proto => @options[:request],
+                     :caller => request_caller }
 
-          return ::Protobuf::Socketrpc::Request.new(
-            :service_name => @options[:service].name,
-            :method_name => @options[:method].to_s,
-            :request_proto => @options[:request]
-          ).serialize_to_string
+          return ::Protobuf::Socketrpc::Request.new(fields).serialize_to_string
         rescue => e
           fail(:INVALID_REQUEST_PROTO, "Could not set request proto: #{e.message}")
         end
 
         def setup_connection
           initialize_stats
-          @request_data = rpc_request_data
+          @request_data = request_bytes
         end
 
         def succeed(response)
@@ -129,7 +133,7 @@ module Protobuf
           complete
         end
 
-        def validate_request_type
+        def validate_request_type!
           unless @options[:request].class == @options[:request_type]
             expected = @options[:request_type].name
             actual = @options[:request].class.name
@@ -138,13 +142,13 @@ module Protobuf
         end
 
         def verify_callbacks
-          if !any_callbacks?
+          unless any_callbacks?
             log_debug { sign_message("No callbacks set, using data_callback") }
             @success_cb = @failure_cb = self.method(:data_callback)
           end
         end
 
-        def verify_options
+        def verify_options!
           # Verify the options that are necessary and merge them in
           [:service, :method, :host, :port].each do |opt|
             fail(:RPC_ERROR, "Invalid client connection configuration. #{opt} must be a defined option.") if @options[opt].nil?

data/lib/protobuf/rpc/connectors/em_client.rb

@@ -17,7 +17,7 @@ module Protobuf
           @failure_cb = failure_cb
           @options = DEFAULT_OPTIONS.merge(options)
           @response_buffer = ::Protobuf::Rpc::Buffer.new(:read)
-          verify_options
+          verify_options!
 
           log_debug { sign_message("Client Initialized: #{options.inspect}") }
         rescue => e

data/lib/protobuf/rpc/rpc.pb.rb

@@ -1,78 +1,6 @@
-### Generated by rprotoc. DO NOT EDIT!
-### <proto file: rpc.proto>
-# // Copyright (c) 2009 Shardul Deo
-# //
-# // Permission is hereby granted, free of charge, to any person obtaining a copy
-# // of this software and associated documentation files (the "Software"), to deal
-# // in the Software without restriction, including without limitation the rights
-# // to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
-# // copies of the Software, and to permit persons to whom the Software is
-# // furnished to do so, subject to the following conditions:
-# //
-# // The above copyright notice and this permission notice shall be included in
-# // all copies or substantial portions of the Software.
-# //
-# // THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
-# // IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
-# // FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
-# // AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
-# // LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
-# // OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
-# // THE SOFTWARE.
-# 
-# // Author: Shardul Deo
-# //
-# // Protobufs needed for socket rpcs.
-# 
-# package protobuf.socketrpc;
-# 
-# message Request {
-# 
-#   // RPC service full name
-#   required string service_name = 1;
-#   
-#   // RPC method name
-#   required string method_name = 2;
-#   
-#   // RPC request proto
-#   required bytes request_proto = 3;
-# }
-# 
-# message Response {
-# 
-#   // RPC response proto
-#   optional bytes response_proto = 1;
-#   
-#   // Error, if any
-#   optional string error = 2;
-#   
-#   // Was callback invoked
-#   optional bool callback = 3 [default = false];
-#   
-#   // Error Reason
-#   optional ErrorReason error_reason = 4;
-# }
-# 
-# // Possible error reasons
-# // The server-side errors are returned in the response from the server.
-# // The client-side errors are returned by the client-side code when it doesn't 
-# // have a response from the server.
-# enum ErrorReason {
-# 
-#   // Server-side errors
-#   BAD_REQUEST_DATA = 0; // Server received bad request data
-#   BAD_REQUEST_PROTO = 1; // Server received bad request proto
-#   SERVICE_NOT_FOUND = 2; // Service not found on server
-#   METHOD_NOT_FOUND = 3; // Method not found on server
-#   RPC_ERROR = 4; // Rpc threw exception on server
-#   RPC_FAILED = 5; // Rpc failed on server
-#   
-#   // Client-side errors (these are returned by the client-side code)
-#   INVALID_REQUEST_PROTO = 6; // Rpc was called with invalid request proto
-#   BAD_RESPONSE_PROTO = 7; // Server returned a bad response proto
-#   UNKNOWN_HOST = 8; // Could not find supplied host
-#   IO_ERROR = 9; // I/O error while communicating with server
-# }
+##
+# This file is auto-generated. DO NOT EDIT!
+#
 require 'protobuf/message'
 
 module Protobuf
@@ -91,28 +19,35 @@ module Protobuf
     ##
     # Enum Values
     #
-    ::Protobuf::Socketrpc::ErrorReason.define :BAD_REQUEST_DATA, 0
-    ::Protobuf::Socketrpc::ErrorReason.define :BAD_REQUEST_PROTO, 1
-    ::Protobuf::Socketrpc::ErrorReason.define :SERVICE_NOT_FOUND, 2
-    ::Protobuf::Socketrpc::ErrorReason.define :METHOD_NOT_FOUND, 3
-    ::Protobuf::Socketrpc::ErrorReason.define :RPC_ERROR, 4
-    ::Protobuf::Socketrpc::ErrorReason.define :RPC_FAILED, 5
-    ::Protobuf::Socketrpc::ErrorReason.define :INVALID_REQUEST_PROTO, 6
-    ::Protobuf::Socketrpc::ErrorReason.define :BAD_RESPONSE_PROTO, 7
-    ::Protobuf::Socketrpc::ErrorReason.define :UNKNOWN_HOST, 8
-    ::Protobuf::Socketrpc::ErrorReason.define :IO_ERROR, 9
+    class ErrorReason
+      define :BAD_REQUEST_DATA, 0
+      define :BAD_REQUEST_PROTO, 1
+      define :SERVICE_NOT_FOUND, 2
+      define :METHOD_NOT_FOUND, 3
+      define :RPC_ERROR, 4
+      define :RPC_FAILED, 5
+      define :INVALID_REQUEST_PROTO, 6
+      define :BAD_RESPONSE_PROTO, 7
+      define :UNKNOWN_HOST, 8
+      define :IO_ERROR, 9
+    end
     
     ##
     # Message Fields
     #
-    ::Protobuf::Socketrpc::Request.required(::Protobuf::Field::StringField, :service_name, 1)
-    ::Protobuf::Socketrpc::Request.required(::Protobuf::Field::StringField, :method_name, 2)
-    ::Protobuf::Socketrpc::Request.required(::Protobuf::Field::BytesField, :request_proto, 3)
+    class Request
+      required ::Protobuf::Field::StringField, :service_name, 1
+      required ::Protobuf::Field::StringField, :method_name, 2
+      optional ::Protobuf::Field::BytesField, :request_proto, 3
+      optional ::Protobuf::Field::StringField, :caller, 4
+    end
     
-    ::Protobuf::Socketrpc::Response.optional(::Protobuf::Field::BytesField, :response_proto, 1)
-    ::Protobuf::Socketrpc::Response.optional(::Protobuf::Field::StringField, :error, 2)
-    ::Protobuf::Socketrpc::Response.optional(::Protobuf::Field::BoolField, :callback, 3, :default => false)
-    ::Protobuf::Socketrpc::Response.optional(::Protobuf::Socketrpc::ErrorReason, :error_reason, 4)
+    class Response
+      optional ::Protobuf::Field::BytesField, :response_proto, 1
+      optional ::Protobuf::Field::StringField, :error, 2
+      optional ::Protobuf::Field::BoolField, :callback, 3, :default => false
+      optional ::Protobuf::Socketrpc::ErrorReason, :error_reason, 4
+    end
     
   end
 end

data/lib/protobuf/rpc/server.rb

@@ -14,7 +14,6 @@ module Protobuf
         @request = ::Protobuf::Socketrpc::Request.new
         @response = ::Protobuf::Socketrpc::Response.new
         @stats = Protobuf::Rpc::Stat.new(:SERVER)
-        set_peer
       end
 
       def disable_gc!
@@ -25,12 +24,11 @@ module Protobuf
         ::GC.enable && ::GC.start if ::Protobuf.gc_pause_server_request?
       end
 
-      # no-op, implemented by including class if desired.
-      def set_peer; end
-
       # Invoke the service method dictated by the proto wrapper request object
       def handle_client
         parse_request_from_buffer
+        @stats.client = @request.caller
+
         @dispatcher = ::Protobuf::Rpc::ServiceDispatcher.new(@request)
         @stats.dispatcher = @dispatcher
 

data/lib/protobuf/rpc/servers/evented/server.rb

@@ -31,10 +31,6 @@ module Protobuf
           log_debug { sign_message("sending data: #{response_buffer.inspect}") }
           super(response_buffer.write)
         end
-
-        def set_peer
-          @stats.client = ::Socket.unpack_sockaddr_in(get_peername)
-        end
       end
     end
   end

data/lib/protobuf/rpc/servers/socket/server.rb

@@ -74,7 +74,7 @@ module Protobuf
                   client, sockaddr = @server.accept
                   @listen_fds << client
                 else
-                  if !@working.include?(client)
+                  unless @working.include?(client)
                     @working << @listen_fds.delete(client)
                     log_debug { sign_message("Working")  }
                     @threads << { :thread => new_worker(client), :socket => client }

data/lib/protobuf/rpc/servers/socket/worker.rb

@@ -52,10 +52,6 @@ module Protobuf
           @_log_signature ||= "server-#{self.class}-#{object_id}"
         end
 
-        def set_peer
-          @stats.client = ::Socket.unpack_sockaddr_in(@socket.getpeername)
-        end
-
         def socket_writable?
           ! @socket.nil? && ! @socket.closed?
         end

data/lib/protobuf/rpc/servers/zmq/worker.rb

@@ -29,7 +29,7 @@ module Protobuf
         def handle_request(socket)
           @request_data = ''
           zmq_error_check(socket.recv_string(@request_data))
-          log_debug { sign_message("handling request") } if !@request_data.nil?
+          log_debug { sign_message("handling request") } unless @request_data.nil?
         end
 
         def run

data/lib/protobuf/rpc/service.rb

@@ -15,8 +15,6 @@ module Protobuf
       include ::Protobuf::Logger::LogMethods
 
 
-      attr_reader :response, :rpc
-
       DEFAULT_HOST = '127.0.0.1'.freeze
       DEFAULT_PORT = 9399
 
@@ -102,23 +100,26 @@ module Protobuf
       # Instance Methods
       #
 
+      attr_reader :response, :method_name, :client_host
+
       # Initialize a service with the rpc endpoint name and the bytes
       # for the request.
-      def initialize(rpc, request_bytes)
-        @rpc = rpc
-        @request_bytes = request_bytes
+      def initialize(method_name, request_bytes, client_host = nil)
+        @method_name = method_name
+        @client_host = client_host
+        @_request_bytes = request_bytes
       end
 
       # Register a failure callback for use when rpc_failed is invoked.
       #
       def on_rpc_failed(callable)
-        @rpc_failed_callback ||= callable
+        @_rpc_failed_callback ||= callable
       end
 
       # Response object for this rpc cycle. Not assignable.
       #
       def response
-        @_response ||= rpcs[@rpc].response_type.new
+        @_response ||= response_type.new
       end
 
       # Convenience method to get back to class method.
@@ -144,18 +145,26 @@ module Protobuf
 
       private
 
+      def response_type
+        @_response_type ||= rpcs[@method_name].response_type
+      end
+
       # Request object for this rpc cycle. Not assignable.
       #
       def request
-        @_request ||= if @request_bytes.present?
-            rpcs[@rpc].request_type.new.parse_from_string(@request_bytes)
+        @_request ||= if @_request_bytes.present?
+            request_type.new.parse_from_string(@_request_bytes)
           else
-            rpcs[@rpc].request_type.new
+            request_type.new
           end
       rescue => e
         raise BadRequestProto, "Unable to parse request: #{e.message}"
       end
 
+      def request_type
+        @_request_type ||= rpcs[@method_name].request_type
+      end
+
       # Sugar to make an rpc method feel like a controller method.
       # If this method is not called, the response will be the memoized
       # object returned by the response reader.
@@ -165,10 +174,22 @@ module Protobuf
       end
       alias_method :return_from_whence_you_came, :respond_with
 
+      # Renamed attribute from prior implementaiton due to lack of clarity
+      # in what the variable contained. DEPRECATED.
+      def rpc
+        if ::Protobuf.print_deprecation_warnings?
+          $stderr.puts <<-ERROR
+            [WARNING] Service#rpc method has been deprecated
+                      and will be removed in a future version of protobuf.
+          ERROR
+        end
+        @method_name
+      end
+
       # Automatically fail a service method.
       #
       def rpc_failed(message)
-        @rpc_failed_callback.call(message)
+        @_rpc_failed_callback.call(message)
       end
 
     end

data/lib/protobuf/rpc/service_dispatcher.rb

@@ -74,7 +74,7 @@ module Protobuf
         request_proto = outer_request.has_field?(:request_proto) ? outer_request.request_proto : nil
 
         if service_klass.rpc_method?(method_name)
-          self.service = service_klass.new(method_name, request_proto)
+          self.service = service_klass.new(method_name, request_proto, outer_request.caller)
           self.callable_method = service.callable_rpc_method(method_name)
           self.definition = service.rpcs[method_name]
         else

data/lib/protobuf/rpc/stat.rb

@@ -14,12 +14,12 @@ module Protobuf
         start
       end
 
-      def client=(peer)
-        @client = {:port => peer[0], :ip => peer[1]}
+      def client=(client_host)
+        @client = client_host
       end
 
       def client
-        @client ? "#{@client[:ip]}:#{@client[:port]}" : nil
+        @client || nil
       end
 
       def method_name
@@ -47,7 +47,7 @@ module Protobuf
       end
 
       def stop
-        start if !@start_time
+        start unless @start_time
         @end_time ||= Time.now
       end
 

data/lib/protobuf/version.rb

@@ -1,4 +1,4 @@
 module Protobuf
-  VERSION = '2.5.0'
+  VERSION = '2.5.1'
   PROTOC_VERSION = '2.4.1'
 end

data/proto/rpc.proto

@@ -18,56 +18,45 @@
 // OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
 // THE SOFTWARE.
 
-// Author: Shardul Deo
+// Authors: Shardul Deo, BJ Neilsen
 //
 // Protobufs needed for socket rpcs.
 
 package protobuf.socketrpc;
 
-message Request {
-
-  // RPC service full name
-  required string service_name = 1;
-  
-  // RPC method name
-  required string method_name = 2;
-  
-  // RPC request proto
-  optional bytes request_proto = 3;
+message Request
+{
+  required string service_name = 1;             // Fully- qualified Service class name
+  required string method_name = 2;              // Service method to invoke
+  optional bytes request_proto = 3;             // Serialized request bytes
+  optional string caller = 4;                   // Calling hostname or address
 }
 
-message Response {
-
-  // RPC response proto
-  optional bytes response_proto = 1;
-  
-  // Error, if any
-  optional string error = 2;
-  
-  // Was callback invoked
-  optional bool callback = 3 [default = false];
-  
-  // Error Reason
-  optional ErrorReason error_reason = 4;
+message Response
+{
+  optional bytes response_proto = 1;            // Serialized response
+  optional string error = 2;                    // Error message, if any
+  optional bool callback = 3 [default = false]; // Was callback invoked (not sure what this is for)
+  optional ErrorReason error_reason = 4;        // Error Reason
 }
 
 // Possible error reasons
 // The server-side errors are returned in the response from the server.
-// The client-side errors are returned by the client-side code when it doesn't 
+// The client-side errors are returned by the client-side code when it doesn't
 // have a response from the server.
-enum ErrorReason {
-
+enum ErrorReason
+{
   // Server-side errors
-  BAD_REQUEST_DATA = 0; // Server received bad request data
-  BAD_REQUEST_PROTO = 1; // Server received bad request proto
-  SERVICE_NOT_FOUND = 2; // Service not found on server
-  METHOD_NOT_FOUND = 3; // Method not found on server
-  RPC_ERROR = 4; // Rpc threw exception on server
-  RPC_FAILED = 5; // Rpc failed on server
-  
+  BAD_REQUEST_DATA = 0;                         // Server received bad request data
+  BAD_REQUEST_PROTO = 1;                        // Server received bad request proto
+  SERVICE_NOT_FOUND = 2;                        // Service not found on server
+  METHOD_NOT_FOUND = 3;                         // Method not found on server
+  RPC_ERROR = 4;                                // Rpc threw exception on server
+  RPC_FAILED = 5;                               // Rpc failed on server
+
   // Client-side errors (these are returned by the client-side code)
-  INVALID_REQUEST_PROTO = 6; // Rpc was called with invalid request proto
-  BAD_RESPONSE_PROTO = 7; // Server returned a bad response proto
-  UNKNOWN_HOST = 8; // Could not find supplied host
-  IO_ERROR = 9; // I/O error while communicating with server
+  INVALID_REQUEST_PROTO = 6;                    // Rpc was called with invalid request proto
+  BAD_RESPONSE_PROTO = 7;                       // Server returned a bad response proto
+  UNKNOWN_HOST = 8;                             // Could not find supplied host
+  IO_ERROR = 9;                                 // I/O error while communicating with server
 }

data/spec/lib/protobuf/rpc/connectors/common_spec.rb

@@ -10,20 +10,23 @@ describe Protobuf::Rpc::Connectors::Common do
     end
   end
 
-  subject{ @subject ||= common_class.new({}) }
+  let(:subject_options) { {} }
+
+  subject { @subject ||= common_class.new(subject_options) }
 
   context "API" do
-    specify{ subject.respond_to?(:any_callbacks?).should be_true }
-    specify{ subject.respond_to?(:data_callback).should be_true }
-    specify{ subject.respond_to?(:error).should be_true }
-    specify{ subject.respond_to?(:fail).should be_true }
-    specify{ subject.respond_to?(:complete).should be_true }
-    specify{ subject.respond_to?(:parse_response).should be_true }
-    specify{ subject.respond_to?(:verify_options).should be_true }
-    specify{ subject.respond_to?(:verify_callbacks).should be_true }
+    specify { subject.respond_to?(:any_callbacks?).should be_true }
+    specify { subject.respond_to?(:request_caller).should be_true }
+    specify { subject.respond_to?(:data_callback).should be_true }
+    specify { subject.respond_to?(:error).should be_true }
+    specify { subject.respond_to?(:fail).should be_true }
+    specify { subject.respond_to?(:complete).should be_true }
+    specify { subject.respond_to?(:parse_response).should be_true }
+    specify { subject.respond_to?(:verify_options!).should be_true }
+    specify { subject.respond_to?(:verify_callbacks).should be_true }
   end
 
-  context "#any_callbacks?" do
+  describe "#any_callbacks?" do
 
     [:@complete_cb, :@success_cb, :@failure_cb].each do |cb|
       it "returns true if #{cb} is provided" do
@@ -42,7 +45,19 @@ describe Protobuf::Rpc::Connectors::Common do
 
   end
 
-  context "#data_callback" do
+  describe '#request_caller' do
+    its(:request_caller) { should eq ::Protobuf.client_host }
+
+    context 'when "client_host" option is given to initializer' do
+      let(:hostname) { 'myhost.myserver.com' }
+      let(:subject_options) { { :client_host => hostname } }
+
+      its(:request_caller) { should_not eq ::Protobuf.client_host }
+      its(:request_caller) { should eq hostname }
+    end
+  end
+
+  describe "#data_callback" do
     it "changes state to use the data callback" do
       subject.data_callback("data")
       subject.instance_variable_get(:@used_data_callback).should be_true
@@ -54,7 +69,28 @@ describe Protobuf::Rpc::Connectors::Common do
     end
   end
 
-  context "#verify_callbacks" do
+  describe '#request_bytes' do
+    let(:service) { Test::ResourceService }
+    let(:method) { :find }
+    let(:request) { '' }
+    let(:client_host) { 'myhost.myservice.com' }
+    let(:subject_options) { { :service => service,
+                              :method => method,
+                              :request => request,
+                              :client_host => client_host } }
+
+    let(:expected) { ::Protobuf::Socketrpc::Request.new({ :service_name => service.name,
+                                                          :method_name => 'find',
+                                                          :request_proto => '',
+                                                          :caller => client_host }) }
+
+    before { subject.stub(:validate_request_type!).and_return(true) }
+    before { subject.should_not_receive(:fail) }
+
+    its(:request_bytes) { should eq expected.serialize_to_string }
+  end
+
+  describe "#verify_callbacks" do
 
     it "sets @failure_cb to #data_callback when no callbacks are defined" do
       subject.verify_callbacks

data/spec/lib/protobuf_spec.rb

@@ -3,6 +3,22 @@ require 'protobuf'
 
 describe ::Protobuf do
 
+  describe '.client_host' do
+    after { ::Protobuf.instance_variable_set(:@_client_host, nil) }
+
+    subject { ::Protobuf.client_host }
+
+    context 'when client_host is not pre-configured' do
+      it { should eq `hostname`.chomp }
+    end
+
+    context 'when client_host is pre-configured' do
+      let(:hostname) { 'override.myhost.com' }
+      before { ::Protobuf.client_host = hostname }
+      it { should eq hostname }
+    end
+  end
+
   describe '.connector_type' do
     before { described_class.instance_variable_set(:@_connector_type, nil) }
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: protobuf
 version: !ruby/object:Gem::Version
-  version: 2.5.0
+  version: 2.5.1
   prerelease: 
 platform: ruby
 authors:
@@ -10,11 +10,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-11-28 00:00:00.000000000 Z
+date: 2012-11-29 00:00:00.000000000Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
-  requirement: !ruby/object:Gem::Requirement
+  requirement: &2152959680 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -22,15 +22,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    none: false
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: '0'
+  version_requirements: *2152959680
 - !ruby/object:Gem::Dependency
   name: ffi
-  requirement: !ruby/object:Gem::Requirement
+  requirement: &2152958340 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -38,15 +33,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    none: false
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: '0'
+  version_requirements: *2152958340
 - !ruby/object:Gem::Dependency
   name: multi_json
-  requirement: !ruby/object:Gem::Requirement
+  requirement: &2152956800 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -54,15 +44,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    none: false
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: '0'
+  version_requirements: *2152956800
 - !ruby/object:Gem::Dependency
   name: thor
-  requirement: !ruby/object:Gem::Requirement
+  requirement: &2152954460 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -70,15 +55,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    none: false
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: '0'
+  version_requirements: *2152954460
 - !ruby/object:Gem::Dependency
   name: eventmachine
-  requirement: !ruby/object:Gem::Requirement
+  requirement: &2152953240 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -86,15 +66,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    none: false
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: '0'
+  version_requirements: *2152953240
 - !ruby/object:Gem::Dependency
   name: ffi-rzmq
-  requirement: !ruby/object:Gem::Requirement
+  requirement: &2152951540 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -102,15 +77,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    none: false
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: '0'
+  version_requirements: *2152951540
 - !ruby/object:Gem::Dependency
   name: pry
-  requirement: !ruby/object:Gem::Requirement
+  requirement: &2152950140 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -118,15 +88,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    none: false
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: '0'
+  version_requirements: *2152950140
 - !ruby/object:Gem::Dependency
   name: pry-nav
-  requirement: !ruby/object:Gem::Requirement
+  requirement: &2152948840 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -134,15 +99,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    none: false
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: '0'
+  version_requirements: *2152948840
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: !ruby/object:Gem::Requirement
+  requirement: &2152947240 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -150,15 +110,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    none: false
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: '0'
+  version_requirements: *2152947240
 - !ruby/object:Gem::Dependency
   name: rake-compiler
-  requirement: !ruby/object:Gem::Requirement
+  requirement: &2152946020 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -166,15 +121,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    none: false
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: '0'
+  version_requirements: *2152946020
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: !ruby/object:Gem::Requirement
+  requirement: &2152871420 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -182,15 +132,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    none: false
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: '0'
+  version_requirements: *2152871420
 - !ruby/object:Gem::Dependency
   name: simplecov
-  requirement: !ruby/object:Gem::Requirement
+  requirement: &2152865540 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -198,15 +143,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    none: false
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: '0'
+  version_requirements: *2152865540
 - !ruby/object:Gem::Dependency
   name: yard
-  requirement: !ruby/object:Gem::Requirement
+  requirement: &2152816060 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -214,12 +154,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    none: false
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: '0'
+  version_requirements: *2152816060
 description: Google Protocol Buffers v2.4.1 Serialization and RPC implementation for
   Ruby.
 email:
@@ -402,7 +337,6 @@ files:
 - lib/protobuf/version.rb
 - lib/protobuf/wire_type.rb
 - lib/protobuf/zmq.rb
-- proto/rpc.pb.rb
 - proto/rpc.proto
 - protobuf.gemspec
 - spec/benchmark/tasks.rb
@@ -498,7 +432,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 2566852558843863563
+      hash: -4197432233485309916
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -507,12 +441,91 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 2566852558843863563
+      hash: -4197432233485309916
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.24
+rubygems_version: 1.8.15
 signing_key: 
 specification_version: 3
 summary: Google Protocol Buffers v2.4.1 Serialization and RPC implementation for Ruby.
-test_files: []
+test_files:
+- spec/benchmark/tasks.rb
+- spec/functional/embedded_service_spec.rb
+- spec/functional/evented_server_spec.rb
+- spec/functional/socket_server_spec.rb
+- spec/functional/zmq_server_spec.rb
+- spec/lib/protobuf/cli_spec.rb
+- spec/lib/protobuf/enum_spec.rb
+- spec/lib/protobuf/enum_value_spec.rb
+- spec/lib/protobuf/logger_spec.rb
+- spec/lib/protobuf/message_spec.rb
+- spec/lib/protobuf/rpc/client_spec.rb
+- spec/lib/protobuf/rpc/connector_spec.rb
+- spec/lib/protobuf/rpc/connectors/base_spec.rb
+- spec/lib/protobuf/rpc/connectors/common_spec.rb
+- spec/lib/protobuf/rpc/connectors/socket_spec.rb
+- spec/lib/protobuf/rpc/connectors/zmq_spec.rb
+- spec/lib/protobuf/rpc/servers/evented_server_spec.rb
+- spec/lib/protobuf/rpc/servers/socket_server_spec.rb
+- spec/lib/protobuf/rpc/servers/zmq/broker_spec.rb
+- spec/lib/protobuf/rpc/servers/zmq/server_spec.rb
+- spec/lib/protobuf/rpc/servers/zmq/util_spec.rb
+- spec/lib/protobuf/rpc/servers/zmq/worker_spec.rb
+- spec/lib/protobuf/rpc/service_dispatcher_spec.rb
+- spec/lib/protobuf/rpc/service_filters_spec.rb
+- spec/lib/protobuf/rpc/service_spec.rb
+- spec/lib/protobuf_spec.rb
+- spec/spec_helper.rb
+- spec/support/all.rb
+- spec/support/server.rb
+- spec/support/test/enum.pb.rb
+- spec/support/test/enum.proto
+- spec/support/test/resource.pb.rb
+- spec/support/test/resource.proto
+- spec/support/test/resource_service.rb
+- spec/support/test_app_file.rb
+- spec/support/tolerance_matcher.rb
+- test/data/data.bin
+- test/data/data_source.py
+- test/data/types.bin
+- test/data/types_source.py
+- test/data/unk.png
+- test/proto/addressbook.pb.rb
+- test/proto/addressbook.proto
+- test/proto/addressbook_base.pb.rb
+- test/proto/addressbook_base.proto
+- test/proto/addressbook_ext.pb.rb
+- test/proto/addressbook_ext.proto
+- test/proto/collision.pb.rb
+- test/proto/collision.proto
+- test/proto/ext_collision.pb.rb
+- test/proto/ext_collision.proto
+- test/proto/ext_range.pb.rb
+- test/proto/ext_range.proto
+- test/proto/float_default.proto
+- test/proto/lowercase.pb.rb
+- test/proto/lowercase.proto
+- test/proto/merge.pb.rb
+- test/proto/merge.proto
+- test/proto/nested.pb.rb
+- test/proto/nested.proto
+- test/proto/optional_field.pb.rb
+- test/proto/optional_field.proto
+- test/proto/packed.pb.rb
+- test/proto/packed.proto
+- test/proto/rpc.proto
+- test/proto/types.pb.rb
+- test/proto/types.proto
+- test/test_addressbook.rb
+- test/test_enum_value.rb
+- test/test_extension.rb
+- test/test_lowercase.rb
+- test/test_message.rb
+- test/test_optional_field.rb
+- test/test_packed_field.rb
+- test/test_parse.rb
+- test/test_repeated_types.rb
+- test/test_serialize.rb
+- test/test_standard_message.rb
+- test/test_types.rb
 has_rdoc: 

data/proto/rpc.pb.rb

@@ -1,48 +0,0 @@
-##
-# This file is auto-generated. DO NOT EDIT!
-#
-require 'protobuf/message'
-
-module Protobuf
-  module Socketrpc
-    ##
-    # Enum Classes
-    #
-    class ErrorReason < ::Protobuf::Enum; end
-    
-    ##
-    # Message Classes
-    #
-    class Request < ::Protobuf::Message; end
-    class Response < ::Protobuf::Message; end
-    
-    ##
-    # Enum Values
-    #
-    ::Protobuf::Socketrpc::ErrorReason.define :BAD_REQUEST_DATA, 0
-    ::Protobuf::Socketrpc::ErrorReason.define :BAD_REQUEST_PROTO, 1
-    ::Protobuf::Socketrpc::ErrorReason.define :SERVICE_NOT_FOUND, 2
-    ::Protobuf::Socketrpc::ErrorReason.define :METHOD_NOT_FOUND, 3
-    ::Protobuf::Socketrpc::ErrorReason.define :RPC_ERROR, 4
-    ::Protobuf::Socketrpc::ErrorReason.define :RPC_FAILED, 5
-    ::Protobuf::Socketrpc::ErrorReason.define :INVALID_REQUEST_PROTO, 6
-    ::Protobuf::Socketrpc::ErrorReason.define :BAD_RESPONSE_PROTO, 7
-    ::Protobuf::Socketrpc::ErrorReason.define :UNKNOWN_HOST, 8
-    ::Protobuf::Socketrpc::ErrorReason.define :IO_ERROR, 9
-    
-    
-    ##
-    # Message Fields
-    #
-    ::Protobuf::Socketrpc::Request.required(::Protobuf::Field::StringField, :service_name, 1)
-    ::Protobuf::Socketrpc::Request.required(::Protobuf::Field::StringField, :method_name, 2)
-    ::Protobuf::Socketrpc::Request.optional(::Protobuf::Field::BytesField, :request_proto, 3)
-    
-    ::Protobuf::Socketrpc::Response.optional(::Protobuf::Field::BytesField, :response_proto, 1)
-    ::Protobuf::Socketrpc::Response.optional(::Protobuf::Field::StringField, :error, 2)
-    ::Protobuf::Socketrpc::Response.optional(::Protobuf::Field::BoolField, :callback, 3, :default => false)
-    ::Protobuf::Socketrpc::Response.optional(::Protobuf::Socketrpc::ErrorReason, :error_reason, 4)
-    
-    
-  end
-end