Google Maps eingebettete Links funktionieren manchmal, manchmal nicht (Onebox 404)

Canapin · 2. November 2022 um 11:57

Ich beobachte ein seltsames Verhalten bei Google Maps-Links.

Manchmal werden sie korrekt eingebettet, und manchmal liefert die Onebox-Anfrage einen 404-Fehler. Ich konnte keine konsistente Möglichkeit finden, das Problem nachzustellen, aber es tritt sehr häufig auf. Versuchen Sie einfach, Links einzufügen, die Google Maps-Kamera zu bewegen oder den Zuschnitt zu ändern, und versuchen Sie es erneut, usw.

Ich dachte zunächst an ein Rate-Limit, aber das ist es nicht. Die Links, die funktionieren, und die, die nicht funktionieren, sind hier oder auf meinem Forum identisch. Beispiel:

Hat jemand eine Idee, woher dieses Problem kommt? Von Google? Von Discourse? Sollte es nach Contribute > Bug verschoben werden?

awesomerobot · 21. Februar 2023 um 21:07

Das ist irgendwie seltsam, ich habe gerade das Muster bemerkt… aber ich glaube, es hat etwas mit der Zoomstufe im Link zu tun?

15.5z

15z

16.75z

17z

Ein völlig anderer Ort…

15.38z

15z

Meine Vermutung ist, dass es sich um ein Problem mit der Google Maps Onebox-Engine auf unserer Seite handelt?

Falco · 21. Februar 2023 um 22:02

Tolle Ermittlungsarbeit @awesomerobot !

Es sieht so aus, als ob unser Regex für die Zoomstufe hier keine Punkte zulässt:

github.com/discourse/discourse

lib/onebox/engine/google_maps_onebox.rb

main


      
              break if type != :standard
              sleep 0.1
            end
          end
          
          case type
          when :standard
            # Fallback for map URLs that don't resolve into an easily embeddable old-style URI
            # Roadmaps use a "z" zoomlevel, satellite maps use "m" the horizontal width in meters
            # TODO: tilted satellite maps using "a,y,t"
            match = @url.match(/@(?<lon>[\d.-]+),(?<lat>[\d.-]+),(?<zoom>\d+)(\.\d+)?(?<mz>[mz])/)
            raise "unexpected standard url #{@url}" unless match
            zoom = match[:mz] == "z" ? match[:zoom] : Math.log2(57280048.0 / match[:zoom].to_f).round
            location = "#{match[:lon]},#{match[:lat]}"
            url = "https://maps.google.com/maps?ll=#{location}&z=#{zoom}&output=embed&dg=ntvb"
            url += "&q=#{$1}" if match = @url.match(%r{/place/([^/\?]+)})
            url += "&cid=#{($1 + $2).to_i(16)}" if @url.match(/!3m1!1s0x(\h{16}):0x(\h{16})/)
            @url = url
            @placeholder =
              "https://maps.googleapis.com/maps/api/staticmap?maptype=roadmap&center=#{location}&zoom=#{zoom}&size=690x400&sensor=false"
          when :custom

Eine ziemlich einfache Lösung.

Canapin · 22. Februar 2023 um 00:01

Was für ein Fang

Danke, dass du das herausgefunden hast!

zogstrip · 28. Februar 2023 um 21:36

@loic kannst du dir das kurz ansehen?

loic · 1. März 2023 um 08:07

Sicher, ich schaue mal nach

loic · 1. März 2023 um 11:46

Hier behoben:

github.com/discourse/discourse

FIX: Allow floats for zoom level in Google Maps onebox

main ← loic-fix-google-map-onebox

opened 11:28AM - 01 Mar 23 UTC

Flink

+3 -3

Sometimes we get Maps URL containing a zoom level as a float (17.5z and not 17z)… but this doesn’t work with our current onebox implementation. While Google accepts those float zoom levels, it removes automatically the floating part in the URL (thus when visiting a Maps URL containing 17.5z, the URL will be rewritten shortly after as 17z). When putting a float zoom level in an embedded URL, this actually breaks (Maps API returns a 400 error). This PR addresses the issue by allowing the onebox engine to match on a zoom level expressed as a float but we only keep the integer part, thus rendering properly maps.

zogstrip · 2. März 2023 um 09:39

@awesomerobot’s Beitrag wird neu erstellt und alle Google Maps-Links funktionieren jetzt

Schließe dies.

Thema		Antworten	Aufrufe
Google Maps Onebox Issue Support	4	2325	3. Mai 2019
Google Maps onebox issues Bug	15	3594	1. Januar 2016
Embedding google maps short urls is broken Feature	9	7032	22. März 2015
Embedded google map stopped working Support	10	4733	30. September 2020
Google maps link not embedding anymore Support	2	652	12. September 2022

Google Maps eingebettete Links funktionieren manchmal, manchmal nicht (Onebox 404)

Verwandte Themen