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

Canapin · 2. November 2022 um 11:57

Ich erlebe ein seltsames Verhalten mit Google Maps-Links.

Manchmal werden sie ordnungsgemäß eingebettet, und manchmal gibt die Onebox-Anfrage einen 404 zurück. Ich konnte keine Möglichkeit finden, das Problem konsistent zu reproduzieren, aber es tritt sehr häufig auf. Versuchen Sie einfach, Links einzufügen, die Google Maps-Kamera zu verschieben oder zuzuschneiden, versuchen Sie es erneut usw.

Ich dachte zuerst an eine Ratenbegrenzung, aber das ist es nicht. Die Links, die funktionieren, und die, die es nicht tun, sind hier oder auf meinem Forum dieselben. Beispiel:

Hat jemand eine Idee, woher dieses Problem kommt? Auf Google-Seite? Auf Discourse-Seite? Sollte es nach 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	6	2299	3. Mai 2019
Google Maps onebox issues Bug	16	3501	1. Januar 2016
Embedding google maps short urls is broken Feature	10	6853	25. März 2015
Embedded google map stopped working Support	11	4547	30. Oktober 2020
Google maps link not embedding anymore Support	3	617	12. September 2022

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

Verwandte Themen