Google Maps link incorporati a volte funzionano, a volte no (onebox 404)

Canapin · 2 Novembre 2022, 11:57am

Sto riscontrando un comportamento strano con i link di Google Maps.

A volte si integrano correttamente, altre volte la richiesta onebox restituisce un 404. Non sono riuscito a trovare un modo per riprodurre il problema in modo coerente, ma capita molto spesso. Basta provare a incollare i link, spostare la telecamera o ritagliare su Google Maps, riprovare, ecc.

All’inizio pensavo a un limite di frequenza (rate limit), ma non è così. I link che funzionano e quelli che non funzionano sono gli stessi qui o sul mio forum. Esempio:

Qualcuno ha un’idea da dove possa derivare questo problema? Dal lato di Google? Da quello di Discourse? Dovrebbe essere spostato in Contribute > Bug?

awesomerobot · 21 Febbraio 2023, 9:07pm

Questa è una cosa strana, ho notato per caso lo schema… ma penso che abbia a che fare con il livello di zoom nel link?

15.5z

15z

16.75z

17z

Una posizione completamente diversa…

15.38z

15z

La mia ipotesi è che si tratti di un problema del motore onebox di Google Maps da parte nostra?

Falco · 21 Febbraio 2023, 10:02pm

Ottimo lavoro investigativo @awesomerobot !

Sembra che la nostra espressione regolare per il livello di zoom non permetta punti qui:

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

Una correzione piuttosto semplice.

Canapin · 22 Febbraio 2023, 12:01am

Che bella presa

Grazie per averlo capito!

zogstrip · 28 Febbraio 2023, 9:36pm

@loic puoi dare una rapida occhiata a questo?

loic · 1 Marzo 2023, 8:07am

Certo, ci do un’occhiata

loic · 1 Marzo 2023, 11:46am

Corretto qui:

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 Marzo 2023, 9:39am

Ricostruzione del post di @awesomerobot e tutti i link di Google Maps ora funzionano

Chiusura di questo.

Argomento		Risposte	Visualizzazioni
Google Maps Onebox Issue Support	4	2325	Maggio 3, 2019
Google Maps onebox issues Bug	15	3594	Gennaio 1, 2016
Embedding google maps short urls is broken Feature	9	7033	Marzo 22, 2015
Embedded google map stopped working Support	10	4733	Settembre 30, 2020
Google maps link not embedding anymore Support	2	652	Settembre 12, 2022

Google Maps link incorporati a volte funzionano, a volte no (onebox 404)

Argomenti correlati