google_maps_service 0.1.0 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 8194de5bd5bef78a91fdf5f9933fed905eea1742
-  data.tar.gz: 8f93d0c8990a0b1ef80317e50d5e067a951f8a1d
+  metadata.gz: 03d75b9918a795abfabe46f98659846b37979be9
+  data.tar.gz: 7a501e622f0d8898ad38b65b30e9957fce151aad
 SHA512:
-  metadata.gz: 1cb6043a5dc3f03730d7cb280fe7b6925e15d2d84ab7b9bbb97a73f2c370cd6e7fb3ee65c1e3927b8a4143560943e71689c9920a1c1e26bbdcb9d275dd65fe55
-  data.tar.gz: 91a08cee378d8ce7d9c03e6a1e4212c16ff7da6333a2181d1da90478f7bb93b0f73082656f250bdbb16b3e1cc4a749d25263f4aa85090cecb57a63d28d02bc56
+  metadata.gz: b67c6aac29d022afc1fb48106c436cfcec7f3370d1de7e7a5eb63f339ebe52ef198cd4f7e6797a2865ff7ea1566165ad40f6c413e92de58ccd27bc82140e965b
+  data.tar.gz: 972a7298efc5729bf9a1cba1f617a9dfdee1096fcc3056dcad1b705d792df8230e5fd48caed3009aeee552594a26c1ad00f2df3a362feee2898d5232fa40cf37

data/.travis.yml

@@ -1,3 +1,5 @@
 language: ruby
 rvm:
-  - 2.2.1
+  - 2.2
+  - 2.1
+  - 2.0.0

data/README.md

@@ -1,6 +1,8 @@
 # Ruby Client for Google Maps Services
 
-*This is porting of [Python Client for Google Maps Services](https://github.com/googlemaps/google-maps-services-python). All Google Maps Service APIs are supported, but some features (e.g: auto retry) are not supported right now.*
+[![Gem Version](https://badge.fury.io/rb/google_maps_service.svg)](http://badge.fury.io/rb/google_maps_service) [![Build Status](https://travis-ci.org/edwardsamuel/google-maps-services-ruby.svg?branch=master)](https://travis-ci.org/edwardsamuel/google-maps-services-ruby) [![Dependency Status](https://gemnasium.com/edwardsamuel/google-maps-services-ruby.svg)](https://gemnasium.com/edwardsamuel/google-maps-services-ruby) [![Code Climate](https://codeclimate.com/github/edwardsamuel/google-maps-services-ruby/badges/gpa.svg)](https://codeclimate.com/github/edwardsamuel/google-maps-services-ruby) [![Coverage Status](https://coveralls.io/repos/edwardsamuel/google-maps-services-ruby/badge.svg?branch=master&service=github)](https://coveralls.io/github/edwardsamuel/google-maps-services-ruby?branch=master)
+
+*This is porting of [Python Client for Google Maps Services](https://github.com/googlemaps/google-maps-services-python). All Google Maps Service APIs are supported, but some features (e.g: rate limiting) are not supported right now.*
 
 ## Description
 
@@ -129,4 +131,4 @@ For more usage examples, check out [the tests](spec/).
 [Time Zone API]: https://developers.google.com/maps/documentation/timezone/
 [Roads API]: https://developers.google.com/maps/documentation/roads/
 
-[issues]: https://github.com/googlemaps/google-maps-services-python/issues
+[issues]: https://github.com/edwardsamuel/google-maps-services-ruby/issues

data/google_maps_service.gemspec

@@ -19,8 +19,9 @@ Gem::Specification.new do |spec|
 
   spec.add_runtime_dependency 'multi_json', '~> 1.11'
   spec.add_runtime_dependency 'hurley', '~> 0.1'
+  spec.add_runtime_dependency 'retriable', '~> 2.0.2'
   spec.add_runtime_dependency 'ruby-hmac', '~> 0.4.0'
-  spec.add_development_dependency 'bundler', '~> 1.8'
+  spec.add_development_dependency 'bundler', '~> 1.7'
   spec.add_development_dependency 'rake', '~> 10.0'
   spec.add_development_dependency 'yard', '~> 0.8.7.6'
   spec.add_development_dependency 'rspec', '~> 3.3'

data/lib/google_maps_service.rb

@@ -1,6 +1,6 @@
 module GoogleMapsService
   class << self
-    attr_accessor :key, :client_id, :client_secret, :ssl, :connection_middleware
+    attr_accessor :key, :client_id, :client_secret, :connect_timeout, :read_timeout, :retry_timeout
 
     def configure
       yield self

data/lib/google_maps_service/client.rb

@@ -1,12 +1,13 @@
 require 'uri'
 require 'hurley'
 require 'multi_json'
+require 'retriable'
 
 module GoogleMapsService
   class Client
     USER_AGENT = "GoogleGeoApiClientRuby/#{GoogleMapsService::VERSION}"
     DEFAULT_BASE_URL = "https://maps.googleapis.com"
-    RETRIABLE_STATUSES = [500, 503, 504]
+    RETRIABLE_ERRORS = [GoogleMapsService::Error::ServerError, GoogleMapsService::Error::RateLimitError]
 
     include GoogleMapsService::Directions
     include GoogleMapsService::DistanceMatrix
@@ -28,10 +29,27 @@ module GoogleMapsService
     # @return [String]
     attr_reader :client_secret
 
+    # Connection timeout for HTTP requests, in seconds.
+    # You should specify read_timeout in addition to this option.
+    # @return [Integer]
+    attr_reader :connect_timeout
+
+    # Read timeout for HTTP requests, in seconds.
+    # You should specify connect_timeout in addition to this
+    # @return [Integer]
+    attr_reader :read_timeout
+
+    # Timeout across multiple retriable requests, in seconds.
+    # @return [Integer]
+    attr_reader :retry_timeout
+
     def initialize(options={})
       @key = options[:key] || GoogleMapsService.key
       @client_id = options[:client_id] || GoogleMapsService.client_id
       @client_secret = options[:client_secret] || GoogleMapsService.client_secret
+      @connect_timeout = options[:connect_timeout] || GoogleMapsService.connect_timeout
+      @read_timeout = options[:read_timeout] || GoogleMapsService.read_timeout
+      @retry_timeout = options[:retry_timeout] || GoogleMapsService.retry_timeout || 60
     end
 
     # Get the current HTTP client
@@ -47,18 +65,23 @@ module GoogleMapsService
     def new_client
       client = Hurley::Client.new
       client.request_options.query_class = Hurley::Query::Flat
+      client.request_options.timeout = @read_timeout if @read_timeout
+      client.request_options.open_timeout = @connect_timeout if @connect_timeout
       client.header[:user_agent] = USER_AGENT
       client
     end
 
     def get(path, params, base_url: DEFAULT_BASE_URL, accepts_client_id: true, custom_response_decoder: nil)
       url = base_url + generate_auth_url(path, params, accepts_client_id)
-      response = client.get url
 
-      if custom_response_decoder
-        return custom_response_decoder.call(response)
+      Retriable.retriable timeout: @retry_timeout,
+                          on: RETRIABLE_ERRORS do |try|
+        response = client.get url
+        if custom_response_decoder
+          return custom_response_decoder.call(response)
+        end
+        return decode_response_body(response)
       end
-      return decode_response_body(response)
     end
 
     # Extract and parse body response as hash. Throw an error if there is something wrong with the response.

data/lib/google_maps_service/convert.rb

@@ -31,18 +31,9 @@ module GoogleMapsService
     # @return [Array] Pair of lat and lng array.
     def normalize_latlng(arg)
       if arg.kind_of?(Hash)
-          if arg.has_key?(:lat) and arg.has_key?(:lng)
-              return arg[:lat], arg[:lng]
-          end
-          if arg.has_key?(:latitude) and arg.has_key?(:longitude)
-              return arg[:latitude], arg[:longitude]
-          end
-          if arg.has_key?("lat") and arg.has_key?("lng")
-              return arg["lat"], arg["lng"]
-          end
-          if arg.has_key?("latitude") and arg.has_key?("longitude")
-              return arg["latitude"], arg["longitude"]
-          end
+        lat = arg[:lat] || arg[:latitude] || arg["lat"] || arg["latitude"]
+        lng = arg[:lng] || arg[:longitude] || arg["lng"] || arg["longitude"]
+        return lat, lng
       elsif arg.kind_of?(Array)
         return arg[0], arg[1]
       end
@@ -137,16 +128,34 @@ module GoogleMapsService
     # @return [String]
     def bounds(arg)
       if arg.kind_of?(Hash)
-        if arg.has_key?("southwest") && arg.has_key?("northeast")
-          return "#{latlng(arg["southwest"])}|#{latlng(arg["northeast"])}"
-        elsif arg.has_key?(:southwest) && arg.has_key?(:northeast)
-          return "#{latlng(arg[:southwest])}|#{latlng(arg[:northeast])}"
-        end
+        southwest = arg[:southwest] || arg["southwest"]
+        northeast = arg[:northeast] || arg["northeast"]
+        return "#{latlng(southwest)}|#{latlng(northeast)}"
       end
 
       raise ArgumentError, "Expected a bounds (southwest/northeast) Hash, but got #{arg.class}"
     end
 
+    # Converts an array of waypoints (path) to the format expected by the Google Maps
+    # server.
+    #
+    # Accept two representation of waypoint:
+    #
+    # 1. String: Name of place or comma-separated lat/lon pair.
+    # 2. Hash/Array: Lat/lon pair.
+    #
+    # @param [Array, String, Hash] waypoints Path.
+    #
+    # @return [String]
+    def waypoints(waypoints)
+      if waypoints.kind_of?(Array) and waypoints.length == 2 and waypoints[0].kind_of?(Numeric) and waypoints[1].kind_of?(Numeric)
+        waypoints = [waypoints]
+      end
+
+      waypoints = as_list(waypoints)
+      return join_list('|', waypoints.map { |k| k.kind_of?(String) ? k : latlng(k) })
+    end
+
     # Decodes a Polyline string into a list of lat/lng hash.
     #
     # See the developer docs for a detailed description of this encoding:

data/lib/google_maps_service/directions.rb

@@ -1,3 +1,5 @@
+require_relative './validator'
+
 module GoogleMapsService
 
   # Performs requests to the Google Maps Directions API.
@@ -47,14 +49,7 @@ module GoogleMapsService
         destination: _convert_waypoint(destination)
       }
 
-      if mode
-        # NOTE(broady): the mode parameter is not validated by the Maps API
-        # server. Check here to prevent silent failures.
-        unless ["driving", "walking", "bicycling", "transit"].include?(mode)
-          raise ArgumentError, "Invalid travel mode."
-        end
-        params[:mode] = mode
-      end
+      params[:mode] = GoogleMapsService::Validator.travel_mode(mode) if mode
 
       if waypoints
         waypoints = GoogleMapsService::Convert.as_list(waypoints)
@@ -64,8 +59,8 @@ module GoogleMapsService
         params[:waypoints] = GoogleMapsService::Convert.join_list("|", waypoints)
       end
 
-      params[:alternatives] = "true" if alternatives
-      params[:avoid] = GoogleMapsService::Convert.join_list("|", avoid) if avoid
+      params[:alternatives] = 'true' if alternatives
+      params[:avoid] = GoogleMapsService::Convert.join_list('|', avoid) if avoid
       params[:language] = language if language
       params[:units] = units if units
       params[:region] = region if region
@@ -73,13 +68,13 @@ module GoogleMapsService
       params[:arrival_time] = GoogleMapsService::Convert.time(arrival_time) if arrival_time
 
       if departure_time and arrival_time
-        raise ArgumentError, "Should not specify both departure_time and arrival_time."
+        raise ArgumentError, 'Should not specify both departure_time and arrival_time.'
       end
 
       params[:transit_mode] = GoogleMapsService::Convert.join_list("|", transit_mode) if transit_mode
       params[:transit_routing_preference] = transit_routing_preference if transit_routing_preference
 
-      return get("/maps/api/directions/json", params)[:routes]
+      return get('/maps/api/directions/json', params)[:routes]
     end
 
     private

data/lib/google_maps_service/distance_matrix.rb

@@ -1,3 +1,5 @@
+require_relative './validator'
+
 module GoogleMapsService
 
   # Performs requests to the Google Maps Distance Matrix API.
@@ -39,47 +41,26 @@ module GoogleMapsService
                         departure_time: nil, arrival_time: nil, transit_mode: nil,
                         transit_routing_preference: nil)
       params = {
-        origins: _convert_path(origins),
-        destinations: _convert_path(destinations)
+        origins: GoogleMapsService::Convert.waypoints(origins),
+        destinations: GoogleMapsService::Convert.waypoints(destinations)
       }
 
-      if mode
-        # NOTE(broady): the mode parameter is not validated by the Maps API
-        # server. Check here to prevent silent failures.
-        unless ["driving", "walking", "bicycling", "transit"].include?(mode)
-          raise ArgumentError, "Invalid travel mode."
-        end
-        params[:mode] = mode
-      end
-
       params[:language] = language if language
-
-      if avoid
-        unless ["tolls", "highways", "ferries"].include?(avoid)
-          raise ArgumentError, "Invalid route restriction."
-        end
-        params[:avoid] = avoid
-      end
-
+      params[:mode] = GoogleMapsService::Validator.travel_mode(mode) if mode
+      params[:avoid] = GoogleMapsService::Validator.avoid(avoid) if avoid
 
       params[:units] = units if units
-      params[:departure_time] = convert.time(departure_time) if departure_time
-      params[:arrival_time] = convert.time(arrival_time) if arrival_time
+      params[:departure_time] = GoogleMapsService::Convert.time(departure_time) if departure_time
+      params[:arrival_time] = GoogleMapsService::Convert.time(arrival_time) if arrival_time
 
       if departure_time and arrival_time
-        raise ArgumentError, "Should not specify both departure_time and arrival_time."
+        raise ArgumentError, 'Should not specify both departure_time and arrival_time.'
       end
 
-      params[:transit_mode] = convert.join_list("|", transit_mode) if transit_mode
+      params[:transit_mode] = GoogleMapsService::Convert.join_list("|", transit_mode) if transit_mode
       params[:transit_routing_preference] = transit_routing_preference if transit_routing_preference
 
-      return get("/maps/api/distancematrix/json", params)
+      return get('/maps/api/distancematrix/json', params)
     end
-
-    private
-      def _convert_path(waypoints)
-        waypoints = GoogleMapsService::Convert.as_list(waypoints)
-        return GoogleMapsService::Convert.join_list("|", waypoints.map { |k| k.kind_of?(String) ? k : GoogleMapsService::Convert.latlng(k) })
-      end
   end
 end

data/lib/google_maps_service/roads.rb

@@ -27,11 +27,7 @@ module GoogleMapsService
     #
     # :rtype: A list of snapped points.
     def snap_to_roads(path: nil, interpolate: false)
-      if path.kind_of?(Array) and path.length == 2 and not path[0].kind_of?(Array)
-        path = [path]
-      end
-
-      path = _convert_path(path)
+      path = GoogleMapsService::Convert.waypoints(path)
 
       params = {
         path: path
@@ -71,12 +67,7 @@ module GoogleMapsService
     # @return [Hash] a dict with both a list of speed limits and a list of the snapped
     #         points.
     def snapped_speed_limits(path: nil)
-
-      if path.kind_of?(Array) and path.length == 2 and not path[0].kind_of?(Array)
-        path = [path]
-      end
-
-      path = _convert_path(path)
+      path = GoogleMapsService::Convert.waypoints(path)
 
       params = {
         path: path
@@ -89,11 +80,6 @@ module GoogleMapsService
     end
 
     private
-      def _convert_path(paths)
-        paths = GoogleMapsService::Convert.as_list(paths)
-        return GoogleMapsService::Convert.join_list("|", paths.map { |k| k.kind_of?(String) ? k : GoogleMapsService::Convert.latlng(k) })
-      end
-
       # Extracts a result from a Roads API HTTP response.
       def extract_roads_body(response)
         begin
@@ -132,7 +118,7 @@ module GoogleMapsService
         end
 
         unless response.status_code == 200
-          raise GoogleMapsService::Error::HTTPError.new(response)
+          raise GoogleMapsService::Error::ApiError.new(response)
         end
 
         return body

data/lib/google_maps_service/validator.rb

@@ -0,0 +1,23 @@
+require_relative './convert'
+
+module GoogleMapsService
+  module Validator
+    module_function
+
+    def travel_mode(mode)
+      # NOTE(broady): the mode parameter is not validated by the Maps API
+      # server. Check here to prevent silent failures.
+      unless [:driving, :walking, :bicycling, :transit].include?(mode.to_sym)
+        raise ArgumentError, 'Invalid travel mode.'
+      end
+      mode
+    end
+
+    def avoid(avoid)
+      unless [:tolls, :highways, :ferries].include?(avoid.to_sym)
+        raise ArgumentError, 'Invalid route restriction.'
+      end
+      avoid
+    end
+  end
+end

data/lib/google_maps_service/version.rb

@@ -1,3 +1,3 @@
 module GoogleMapsService
-  VERSION = "0.1.0"
+  VERSION = '0.2.0'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: google_maps_service
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.2.0
 platform: ruby
 authors:
 - Edward Samuel Pasaribu
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-09-13 00:00:00.000000000 Z
+date: 2015-09-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: multi_json
@@ -38,6 +38,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '0.1'
+- !ruby/object:Gem::Dependency
+  name: retriable
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 2.0.2
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 2.0.2
 - !ruby/object:Gem::Dependency
   name: ruby-hmac
   requirement: !ruby/object:Gem::Requirement
@@ -58,14 +72,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.8'
+        version: '1.7'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.8'
+        version: '1.7'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -186,6 +200,7 @@ files:
 - lib/google_maps_service/geocoding.rb
 - lib/google_maps_service/roads.rb
 - lib/google_maps_service/time_zone.rb
+- lib/google_maps_service/validator.rb
 - lib/google_maps_service/version.rb
 homepage: https://github.com/edwardsamuel/google-maps-services-ruby
 licenses: