I link incorporati di Google Maps 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 vengono incorporati correttamente e a volte la richiesta di onebox restituisce un 404. Non sono riuscito a trovare un modo per riprodurre il problema in modo coerente, ma succede molto spesso. Prova semplicemente a incollare i link, a spostare la telecamera di Google Maps o a ritagliare, riprova, ecc…

Inizialmente ho pensato a un limite di richieste, 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 di dove provenga questo problema? Lato Google? Lato Discourse? Dovrebbe essere spostato in 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	6	2299	Maggio 3, 2019
Google Maps onebox issues Bug	16	3501	Gennaio 1, 2016
Embedding google maps short urls is broken Feature	10	6853	Marzo 25, 2015
Embedded google map stopped working Support	11	4549	Ottobre 30, 2020
Google maps link not embedding anymore Support	3	617	Settembre 12, 2022

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

Argomenti correlati