google_distance_matrix 0.4.0 → 0.6.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 3ca9d7d93abd6340a3cb5d36d03cbadfda5e4e1c
-  data.tar.gz: 2cb7956404bd763af695eae8c10b81de5db48450
+SHA256:
+  metadata.gz: de97d6fbd5c11056be61c2fbdade541bcd9e04f95dfdc3d3994d25e97b21476b
+  data.tar.gz: 71fb7ba1f8c2e707a95878219ddf99e040892d6d197a1ed5c33bed671191628e
 SHA512:
-  metadata.gz: 2c5364924b1a055ce8c67c1508b219a1fc187ee9ecfa95d683775c21888f1076518bd1617b6a2e5be0020f5ef862e6707341bf8c71fff60e12e011d4dad719b9
-  data.tar.gz: 59c211df2561ea00a1da7858051fb87de4a02e35642faba695852baf974e9e3d30d34c9b710e3078e05a2cf1ea1caefb6411f35a0604a90a6f01c37f9c9af715
+  metadata.gz: 0304ccbedb35ddde62968025032e2b3d83a962ec857d739df35e3645c4b4220e1aa554729876bcc4aa26e0a9db008ff0e67e314c23ddf4bcbef4951f9d0d13c8
+  data.tar.gz: 650669b42ece96bb992ffccc3738f09ab97d81c02129fdcd4ca0007c921e32fbd834126aec5af62074be5f25ad422f88be3ca454d15c47d83cf60e33ba990ede

data/.editorconfig

@@ -0,0 +1,16 @@
+# EditorConfig helps developers define and maintain consistent
+# coding styles between different editors and IDEs
+# editorconfig.org
+
+root = true
+
+[*]
+end_of_line = lf
+charset = utf-8
+trim_trailing_whitespace = true
+insert_final_newline = true
+indent_style = space
+indent_size = 2
+
+[*.{diff,md}]
+trim_trailing_whitespace = false

data/.rubocop.yml

@@ -0,0 +1,6 @@
+Metrics/LineLength:
+  Max: 100
+  
+Metrics/BlockLength:
+  Exclude:
+    - 'spec/**/*_spec.rb'

data/.ruby-version

@@ -1 +1 @@
-2.3.1
+2.5.5

data/.travis.yml

@@ -1,12 +1,10 @@
 before_install:
   - gem update bundler
 rvm:
-  - 2.1.0
-  - 2.1.2
-  - 2.1.5
-  - 2.2.3
-  - 2.3.0
-  - 2.3.1
+  - 2.3.8
+  - 2.4.5
+  - 2.5.5
+  - 2.6.2
   - ruby-head
 matrix:
   allow_failures:

data/CHANGELOG.md

@@ -1,3 +1,43 @@
+## v.0.6.3
+
+* chore: bumped dependency support up to Rails 6.1 (by mintyfresh)
+
+## v.0.6.2
+
+* chore: bumped dependency support up to Rails 6 (by zackchandler)
+
+## v.0.6.1
+
+* fix: when matrix places were built from hashes, passing hashes to route/s_for doesnt't work (by brauliomartinezlm)
+* fix: place comparison was not working with == (by brauliomartinezlm)
+
+## v.0.6.0
+
+* Depend on activemodel & activesupport < 5.3 (by brauliomartinezlm)
+* Tested with Ruby 2.5.
+* Dropped support for Ruby 2.2 and below.
+* Added support for `channel` (by michaelgpearce)
+
+## v.0.5.0
+
+This release contains breaking change where `url` has been renamed to
+`sensitive_url`. A `filtered_url` method is added to make it clear that
+the URL returned is filtered according to configuration while the other one
+will contain sensitive information like key and signature.
+
+The cache key is changed so it no longer uses the URL, but a digest of the URL
+as key. You may set `config.cache_key_transform` to a block passing given url
+through if you don't want this.
+
+* Fixed an issue where read/write to cache used url with sensitive data and
+  and filtered url resulting in cache miss.
+* Instrumentation payload `url` renamed `sensitive_url`.
+* Instrumentation payload added `filtered_url`.
+* Cache key is a digest of the `sensitive_url` so we don't store in cache the
+  sensitive parts of the URL.
+* Digesting cache key is configurable with `cache_key_transform`. It's a callable
+  object expected to take the url and transform it to the key you want.
+
 ## v.0.4.0
 * When mode is `driving` and `departure_time` is set all `route` objects will contain
   `duration_in_traffic_in_seconds` and `duration_in_traffic_text`.

data/Gemfile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 source 'https://rubygems.org'
 
 # Specify your gem's dependencies in google_distance_matrix.gemspec

data/README.md

@@ -11,9 +11,6 @@ pull the distance matrix from Google.
 Once you have the matrix you can fetch all routes from a given
 origin or to a given destination.
 
-The matrix may also be used as a data set for traveling salesman problem,
-but to solve it you may look at <http://ai4r.org/>.
-
 
 
 

data/Rakefile

@@ -1,9 +1,14 @@
-require "bundler/gem_tasks"
+# frozen_string_literal: true
 
-require "rspec/core/rake_task"
+require 'bundler/gem_tasks'
+
+require 'rubocop/rake_task'
+RuboCop::RakeTask.new
+
+require 'rspec/core/rake_task'
 RSpec::Core::RakeTask.new(:spec) do |t|
-  t.rspec_opts = ["-c", "-f progress", "-r ./spec/spec_helper.rb"]
+  t.rspec_opts = ['-c', '-f progress', '-r ./spec/spec_helper.rb']
   t.pattern = 'spec/**/*_spec.rb'
 end
 
-task default: :spec
+task default: %i[rubocop spec]

data/google_distance_matrix.gemspec

@@ -1,30 +1,32 @@
-# coding: utf-8
-lib = File.expand_path('../lib', __FILE__)
+# frozen_String_literal: true
+
+lib = File.expand_path('lib', __dir__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require 'google_distance_matrix/version'
 
 Gem::Specification.new do |spec|
-  spec.name          = "google_distance_matrix"
+  spec.name          = 'google_distance_matrix'
   spec.version       = GoogleDistanceMatrix::VERSION
-  spec.authors       = ["Thorbjørn Hermansen"]
-  spec.email         = ["thhermansen@gmail.com"]
-  spec.description   = %q{Ruby client for The Google Distance Matrix API}
-  spec.summary       = %q{Ruby client for The Google Distance Matrix API}
-  spec.homepage      = ""
-  spec.license       = "MIT"
+  spec.authors       = ['Thorbjørn Hermansen']
+  spec.email         = ['thhermansen@gmail.com']
+  spec.description   = %(Ruby client for The Google Distance Matrix API)
+  spec.summary       = %(Ruby client for The Google Distance Matrix API)
+  spec.homepage      = ''
+  spec.license       = 'MIT'
 
-  spec.files         = `git ls-files`.split($/)
+  spec.files         = `git ls-files`.split($RS)
   spec.executables   = spec.files.grep(%r{^bin/}) { |f| File.basename(f) }
   spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
-  spec.require_paths = ["lib"]
+  spec.require_paths = ['lib']
 
-  spec.add_dependency "activesupport", ">= 3.2.13"
-  spec.add_dependency "activemodel", ">= 3.2.13"
-  spec.add_dependency "google_business_api_url_signer", "~> 0.1.3"
+  spec.add_dependency 'activemodel', '>= 3.2.13', '< 6.2'
+  spec.add_dependency 'activesupport', '>= 3.2.13', '< 6.2'
+  spec.add_dependency 'google_business_api_url_signer', '~> 0.1.3'
 
-  spec.add_development_dependency "bundler"
-  spec.add_development_dependency "rspec", "~> 3.4.0"
-  spec.add_development_dependency "shoulda-matchers", "~> 3.1.1"
-  spec.add_development_dependency "webmock", "~> 2.0.2"
-  spec.add_development_dependency "rake"
+  spec.add_development_dependency 'bundler'
+  spec.add_development_dependency 'rake'
+  spec.add_development_dependency 'rspec', '~> 3.8.0'
+  spec.add_development_dependency 'rubocop', '~> 0.59.2'
+  spec.add_development_dependency 'shoulda-matchers', '~> 4.0.0.rc1'
+  spec.add_development_dependency 'webmock', '~> 3.4.2'
 end

data/lib/google_distance_matrix.rb

@@ -1,29 +1,31 @@
-require "google_distance_matrix/version"
-
-require "cgi"
-require "json"
-require "active_model"
-require "active_support/core_ext/hash"
-require "google_business_api_url_signer"
-
-require "google_distance_matrix/logger"
-require "google_distance_matrix/errors"
-require "google_distance_matrix/configuration"
-require "google_distance_matrix/url_builder"
-require "google_distance_matrix/client"
-require "google_distance_matrix/client_cache"
-require "google_distance_matrix/routes_finder"
-require "google_distance_matrix/matrix"
-require "google_distance_matrix/places"
-require "google_distance_matrix/place"
-require "google_distance_matrix/route"
-require "google_distance_matrix/polyline_encoder"
+# frozen_string_literal: true
+
+require 'google_distance_matrix/version'
+
+require 'cgi'
+require 'json'
+require 'active_model'
+require 'active_support/core_ext/hash'
+require 'google_business_api_url_signer'
+
+require 'google_distance_matrix/logger'
+require 'google_distance_matrix/errors'
+require 'google_distance_matrix/configuration'
+require 'google_distance_matrix/url_builder'
+require 'google_distance_matrix/client'
+require 'google_distance_matrix/client_cache'
+require 'google_distance_matrix/routes_finder'
+require 'google_distance_matrix/matrix'
+require 'google_distance_matrix/places'
+require 'google_distance_matrix/place'
+require 'google_distance_matrix/route'
+require 'google_distance_matrix/polyline_encoder'
 
 require 'google_distance_matrix/railtie' if defined? Rails
 
-
+# Main module for the GoogleDistanceMatrix
 module GoogleDistanceMatrix
-  extend self
+  module_function
 
   def default_configuration
     @default_configuration ||= Configuration.new
@@ -38,4 +40,4 @@ module GoogleDistanceMatrix
   end
 end
 
-require "google_distance_matrix/log_subscriber"
+require 'google_distance_matrix/log_subscriber'

data/lib/google_distance_matrix/client.rb

@@ -1,4 +1,7 @@
+# frozen_string_literal: true
+
 module GoogleDistanceMatrix
+  # HTTP client making request to Google's API
   class Client
     CLIENT_ERRORS = %w[
       INVALID_REQUEST
@@ -6,41 +9,52 @@ module GoogleDistanceMatrix
       OVER_QUERY_LIMIT
       REQUEST_DENIED
       UNKNOWN_ERROR
-    ]
-
-    def get(url, options = {})
+    ].freeze
+
+    # Make a GET request to given URL
+    #
+    # @param url              The URL to Google's API we'll make a request to
+    # @param instrumentation  A hash with instrumentation payload
+    # @param options          Other options we don't care about, for example we'll capture
+    #                         `configuration` option which we are not using, but the ClientCache
+    #                         is using.
+    #
+    # @return Hash with data from parsed response body
+    def get(url, instrumentation: {}, **_options)
       uri = URI.parse url
-      instrumentation = {url: url}.merge(options[:instrumentation] || {})
 
-      response = ActiveSupport::Notifications.instrument "client_request_matrix_data.google_distance_matrix", instrumentation do
+      response = ActiveSupport::Notifications.instrument(
+        'client_request_matrix_data.google_distance_matrix', instrumentation
+      ) do
         Net::HTTP.get_response uri
       end
 
+      handle response, url
+    rescue Timeout::Error => error
+      raise ServerError, error
+    end
+
+    private
+
+    def handle(response, url) # rubocop:disable Metrics/MethodLength
       case response
       when Net::HTTPSuccess
         inspect_for_client_errors! response
       when Net::HTTPRequestURITooLong
-        fail MatrixUrlTooLong.new url, UrlBuilder::MAX_URL_SIZE, response
+        raise MatrixUrlTooLong.new url, UrlBuilder::MAX_URL_SIZE, response
       when Net::HTTPClientError
-        fail ClientError.new response
+        raise ClientError, response
       when Net::HTTPServerError
-        fail ServerError.new response
+        raise ServerError, response
       else # Handle this as a request error for now. Maybe fine tune this more later.
-        fail ServerError.new response
+        raise ServerError, response
       end
-    rescue Timeout::Error => error
-      fail ServerError.new error
     end
 
-
-    private
-
     def inspect_for_client_errors!(response)
-      status = JSON.parse(response.body).fetch "status"
+      status = JSON.parse(response.body).fetch 'status'
 
-      if CLIENT_ERRORS.include? status
-        fail ClientError.new response, status
-      end
+      raise ClientError.new response, status if CLIENT_ERRORS.include? status
 
       response
     end

data/lib/google_distance_matrix/client_cache.rb

@@ -1,9 +1,15 @@
+# frozen_string_literal: true
+
 module GoogleDistanceMatrix
+  # Cached client, which takes care of caching data from Google API
   class ClientCache
     attr_reader :client, :cache
 
-    def self.key(url)
-      url
+    # Returns a cache key for given URL
+    #
+    # @return String
+    def self.key(url, config)
+      config.cache_key_transform.call url
     end
 
     def initialize(client, cache)
@@ -12,7 +18,7 @@ module GoogleDistanceMatrix
     end
 
     def get(url, options = {})
-      cache.fetch self.class.key(url) do
+      cache.fetch self.class.key(url, options.fetch(:configuration)) do
         client.get url, options
       end
     end

data/lib/google_distance_matrix/configuration.rb

@@ -1,3 +1,7 @@
+# frozen_string_literal: true
+
+require 'digest'
+
 module GoogleDistanceMatrix
   # Public: Configuration of matrix and it's request.
   #
@@ -14,20 +18,21 @@ module GoogleDistanceMatrix
       mode avoid units language
       departure_time arrival_time
       transit_mode transit_routing_preference
-      traffic_model
-    ]
+      traffic_model channel
+    ].freeze
 
     API_DEFAULTS = {
-      mode: "driving",
-      units: "metric",
-      traffic_model: "best_guess",
+      mode: 'driving',
+      units: 'metric',
+      traffic_model: 'best_guess',
       use_encoded_polylines: false,
       protocol: 'https',
       lat_lng_scale: 5,
-      filter_parameters_in_logged_url: ['key', 'signature'].freeze
+      filter_parameters_in_logged_url: %w[key signature].freeze,
+      cache_key_transform: ->(url) { Digest::SHA512.new.update(url).to_s }
     }.with_indifferent_access
 
-    attr_accessor *ATTRIBUTES
+    attr_accessor(*ATTRIBUTES)
 
     # The protocol to use, either http or https
     attr_accessor :protocol
@@ -50,22 +55,38 @@ module GoogleDistanceMatrix
     # When logging we filter sensitive parameters
     attr_accessor :filter_parameters_in_logged_url
 
-    validates :mode, inclusion: {in: ["driving", "walking", "bicycling", "transit"]}, allow_blank: true
-    validates :avoid, inclusion: {in: ["tolls", "highways", "ferries", "indoor"]}, allow_blank: true
-    validates :units, inclusion: {in: ["metric", "imperial"]}, allow_blank: true
+    # Callable object which transform given url to key used in cache
+    # @see ClientCache
+    attr_accessor :cache_key_transform
+
+    validates :mode, inclusion: { in: %w[driving walking bicycling transit] }, allow_blank: true
+    validates :avoid, inclusion: { in: %w[tolls highways ferries indoor] }, allow_blank: true
+    validates :units, inclusion: { in: %w[metric imperial] }, allow_blank: true
 
     validates :departure_time, format: /\A(\d+|now)\Z/, allow_blank: true
     validates :arrival_time, numericality: true, allow_blank: true
 
-    validates :transit_mode, inclusion: {in: %w[bus subway train tram rail]}, allow_blank: true
-    validates :transit_routing_preference, inclusion: {in: %w[less_walking fewer_transfers]}, allow_blank: true
-    validates :traffic_model, inclusion: {in: %w[best_guess pessimistic optimistic]}, allow_blank: true
+    validates :transit_mode,
+              inclusion: { in: %w[bus subway train tram rail] },
+              allow_blank: true
+
+    validates :transit_routing_preference,
+              inclusion: { in: %w[less_walking fewer_transfers] },
+              allow_blank: true
+
+    validates :traffic_model,
+              inclusion: { in: %w[best_guess pessimistic optimistic] },
+              allow_blank: true
 
-    validates :protocol, inclusion: {in: ["http", "https"]}, allow_blank: true
+    validates :protocol, inclusion: { in: %w[http https] }, allow_blank: true
 
     def initialize
       API_DEFAULTS.each_pair do |attr_name, value|
-        self[attr_name] = value.dup rescue value
+        self[attr_name] = begin
+                            value.dup
+                          rescue StandardError
+                            value
+                          end
       end
     end
 
@@ -77,7 +98,6 @@ module GoogleDistanceMatrix
       public_send "#{attr_name}=", value
     end
 
-
     private
 
     def array_param
@@ -85,19 +105,14 @@ module GoogleDistanceMatrix
         attr_and_value[1].nil? || param_same_as_api_default?(attr_and_value)
       end
 
-      if google_business_api_client_id.present?
-        out << ['client', google_business_api_client_id]
-      end
-
-      if google_api_key.present?
-        out << ['key', google_api_key]
-      end
+      out << ['client', google_business_api_client_id] if google_business_api_client_id.present?
+      out << ['key', google_api_key] if google_api_key.present?
 
       out
     end
 
     def param_same_as_api_default?(param)
-     API_DEFAULTS[param[0]] == param[1]
+      API_DEFAULTS[param[0]] == param[1]
     end
   end
 end