Les liens intégrés Google Maps fonctionnent parfois, parfois non (onebox 404)

Canapin · Novembre 2, 2022, 11:57

Je rencontre un comportement étrange avec les liens Google Maps.

Parfois, ils s’affichent correctement et parfois la requête onebox renvoie un 404. Je n’ai pas trouvé de moyen de reproduire le problème de manière constante, mais c’est très fréquent. Essayez simplement de coller des liens, de déplacer la caméra de Google Maps ou de recadrer, réessayez, etc.

J’ai d’abord pensé à une limitation de débit, mais ce n’est pas le cas. Les liens qui fonctionnent et ceux qui ne fonctionnent pas sont les mêmes ici ou sur mon forum. Exemple :

Quelqu’un a-t-il une idée de l’origine de ce problème ? Côté Google ? Côté Discourse ? Faut-il déplacer ce sujet dans Bug ?

awesomerobot · Février 21, 2023, 9:07

C’est un peu étrange, j’ai remarqué le schéma par hasard… mais je pense que cela a quelque chose à voir avec le niveau de zoom dans le lien ?

15.5z

15z

16.75z

17z

Un endroit complètement différent…

15.38z

15z

Je suppose que c’est un problème avec le moteur onebox de Google Maps de notre côté ?

Falco · Février 21, 2023, 10:02

Bien joué @awesomerobot !

Il semble que notre regex pour le niveau de zoom ne permette pas les points ici :

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

Une correction assez facile.

Canapin · Février 22, 2023, 12:01

Quelle prise

Merci d’avoir trouvé !

zogstrip · Février 28, 2023, 9:36

@loic peux-tu jeter un coup d’œil rapide à celui-ci ?

loic · Mars 1, 2023, 8:07

Bien sûr, je vais jeter un œil

loic · Mars 1, 2023, 11:46

Corrigé ici :

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 · Mars 2, 2023, 9:39

La publication de @awesomerobot est en cours de reconstruction et tous les liens Google Maps fonctionnent à nouveau

Fermeture de ce ticket.

Sujet		Réponses	Vues
Google Maps Onebox Issue Support	6	2299	Mai 3, 2019
Google Maps onebox issues Bug	16	3501	Janvier 1, 2016
Embedding google maps short urls is broken Feature	10	6853	Mars 25, 2015
Embedded google map stopped working Support	11	4547	Octobre 30, 2020
Google maps link not embedding anymore Support	3	617	Septembre 12, 2022

Les liens intégrés Google Maps fonctionnent parfois, parfois non (onebox 404)

Sujets connexes