implement support for our own python reverse geocoding endpoint

doug · doug · commit ebc096a5f5c5 · 2025-11-08T11:42:45.000-06:00
diff --git a/lib/geocoder/lookup.rb b/lib/geocoder/lookup.rb
@@ -68,7 +68,8 @@ def street_services
         :geoapify,
         :photon,
         :twogis,
-        :pc_miler
+        :pc_miler,
+        :reverse_geocoder
       ]
     end
 
diff --git a/lib/geocoder/lookups/base.rb b/lib/geocoder/lookups/base.rb
@@ -222,6 +222,8 @@ def parse_json(data)
       # Parses a raw search result (returns hash or array).
       #
       def parse_raw_data(raw_data)
+        ap "parsing..."
+        ap raw_data
         parse_json(raw_data)
       end
 
@@ -306,6 +308,10 @@ def make_api_request(query)
               configuration.basic_auth[:password]
             )
           end
+          puts "attemptinmgm client requewst...."
+          ap configuration.http_headers
+          ap client
+          ap req
           client.request(req)
         end
       rescue Timeout::Error
diff --git a/lib/geocoder/lookups/reverse_geocoder.rb b/lib/geocoder/lookups/reverse_geocoder.rb
@@ -0,0 +1,83 @@
+require 'geocoder/lookups/base'
+require 'geocoder/results/reverse_geocoder'
+
+##
+#  Host endpoint: .../reverse?lat=?&lon=?
+#  Host result: returns the python reverse_geocoder result in JSON
+#  See python reverse_geocoder module for more details
+module Geocoder::Lookup
+  class ReverseGeocoder < Base
+
+    def name
+      "Reverse Geocoder"
+    end
+
+    def supported_protocols
+      [:https, :http]
+    end
+
+    private # ---------------------------------------------------------------
+
+    def base_query_url(query)
+      if query.reverse_geocode?
+        "#{protocol}://#{host}/reverse_geocode?"
+      else
+        raise "Forward geocode not supported"
+      end
+    end
+
+    def query_url_params(query)
+      params = {
+      }.merge(super)
+
+      if query.reverse_geocode?
+        lat,lon = query.coordinates
+        params[:lat] = lat
+        params[:lon] = lon
+      else
+        params[:q] = query.sanitized_text
+      end
+      params
+    end
+
+    ##
+    # called by fetch_data
+    def parse_raw_data(raw_data)
+      raw_data.match(/^<html><title>404/) ? nil : super(raw_data)
+    end
+
+    def results(query)
+      return [] unless doc = fetch_data(query)
+      ap doc
+      doc.is_a?(Array) ? doc : [doc]
+      return doc.map{|r| remap_result(r)}
+    end
+
+    def remap_result(r)
+      {
+        lat: r["lat"],
+        lon: r["lon"],
+        city: r["name"],
+        state: r["admin1"],
+        county: r["admin2"],
+        country_code: r["cc"]
+      }
+    end
+
+    def reserved_result(ip)
+      {
+        'ip' => ip,
+        'city' => '',
+        'state' => '',
+        'county' => '',
+        'latitude' => '0',
+        'longitude' => '0',
+        'country_code' => 'RD'
+      }
+    end
+
+    def host
+      configuration[:host] || raise( "Reverse Geocoder requires the host configuration.")
+    end
+  end
+end
diff --git a/lib/geocoder/results/reverse_geocoder.rb b/lib/geocoder/results/reverse_geocoder.rb
@@ -0,0 +1,57 @@
+require 'geocoder/results/base'
+
+module Geocoder::Result
+  class ReverseGeocoder < Base
+
+    def address(format = :full)
+      [
+        city,
+        state,
+        county,
+        [country.presence, country_code.presence].compact.join(" / ")
+      ].compact.join(", ")
+    end
+
+    def county
+      @data[:county]
+    end
+
+    def coordinates
+      [@data[:lat].to_f, @data[:lon].to_f]
+    end
+
+    def city
+      @data[:city]
+    end
+
+    def state
+      @data[:state]
+    end
+
+    def state_code
+      ""
+    end
+
+    def country
+      @data[:country_name]
+    end
+
+    def country_code
+      @data[:country_code]
+    end
+
+    def postal_code
+      @data[:zip]
+    end
+
+    def self.response_attributes
+      %w[country_code ip]
+    end
+
+    response_attributes.each do |a|
+      define_method a do
+        @data[a]
+      end
+    end
+  end
+end

Original file line number	Diff line number	Diff line change
`@@ -68,7 +68,8 @@ def street_services`
`68`	`68`	`:geoapify,`
`69`	`69`	`:photon,`
`70`	`70`	`:twogis,`
`71`		`- :pc_miler`
	`71`	`+ :pc_miler,`
	`72`	`+ :reverse_geocoder`
`72`	`73`	`]`
`73`	`74`	`end`
`74`	`75`