Los enlaces incrustados de Google Maps a veces funcionan y a veces no (onebox 404)

Canapin · 2 Noviembre, 2022 11:57

Me encuentro con un comportamiento extraño con los enlaces de Google Maps.

A veces se incrustan correctamente y, a veces, la solicitud de onebox devuelve un 404. No he encontrado una forma de reproducir el problema de manera consistente, pero ocurre con mucha frecuencia. Simplemente intenta pegar enlaces, mover la cámara de Google Maps o recortar, intenta de nuevo, etc…

Primero pensé en un límite de tasa, pero no lo es. Los enlaces que funcionan y los que no son los mismos aquí o en mi foro. Ejemplo:

¿Alguien tiene una idea de dónde proviene este problema? ¿Del lado de Google? ¿Del lado de Discourse? ¿Debería moverse a Contribute > Bug?

awesomerobot · 21 Febrero, 2023 21:07

Esto es un poco extraño, me acabo de dar cuenta del patrón… pero creo que tiene algo que ver con el nivel de zoom en el enlace?

15.5z

15z

16.75z

17z

Una ubicación completamente diferente…

15.38z

15z

¿Mi suposición es que es un problema del motor de Google Maps onebox de nuestro lado?

Falco · 21 Febrero, 2023 22:02

¡Buena investigación @awesomerobot!

Parece que nuestra expresión regular para el nivel de zoom no permite puntos aquí:

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 solución bastante sencilla.

Canapin · 22 Febrero, 2023 00:01

¡Qué buena captura!

¡Gracias por resolverlo!

zogstrip · 28 Febrero, 2023 21:36

@loic ¿puedes echar un vistazo rápido a este?

loic · 1 Marzo, 2023 08:07

Claro, echaré un vistazo

loic · 1 Marzo, 2023 11:46

Corregido aquí:

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 09:39

Reconstruyendo la publicación de @awesomerobot y todos los enlaces de Google Maps ahora funcionan

Cerrando esto.

Tema		Respuestas	Vistas
Google Maps Onebox Issue Support	4	2325	3 Mayo 2019
Google Maps onebox issues Bug	15	3594	1 Enero 2016
Embedding google maps short urls is broken Feature	9	7032	22 Marzo 2015
Embedded google map stopped working Support	10	4733	30 Septiembre 2020
Google maps link not embedding anymore Support	2	652	12 Septiembre 2022

Los enlaces incrustados de Google Maps a veces funcionan y a veces no (onebox 404)

Temas relacionados