Links incorporados do Google Maps às vezes funcionam, às vezes não (onebox 404)

Canapin · Novembro 2, 2022, 11:57am

Estou enfrentando um comportamento estranho com os links do Google Maps.

Às vezes, eles são incorporados corretamente e, às vezes, a solicitação onebox retorna um erro 404. Não consegui encontrar uma maneira de reproduzir o problema consistentemente, mas isso acontece com muita frequência. Basta tentar colar links, mover a câmera do Google Maps ou ajustar o corte, tentar novamente, etc.

Achei que fosse um limite de taxa (rate limit), mas não é. Os links que funcionam e os que não funcionam são os mesmos aqui ou no meu fórum. Exemplo:

Alguém tem uma ideia de onde vem esse problema? Do lado do Google? Do lado do Discourse? Deveria ser movido para Contribute > Bug?

awesomerobot · Fevereiro 21, 2023, 9:07pm

Isso é meio estranho, acabei de notar o padrão… mas acho que tem a ver com o nível de zoom no link?

15.5z

15z

16.75z

17z

Uma localização totalmente diferente…

15.38z

15z

Minha aposta é que seja um problema no motor do onebox do Google Maps do nosso lado?

Falco · Fevereiro 21, 2023, 10:02pm

Ótima investigação @awesomerobot!

Parece que nossa regex para o nível de zoom não permite pontos aqui:

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

Uma correção bem fácil.

Canapin · Fevereiro 22, 2023, 12:01am

Que pegada

Obrigado por descobrir!

zogstrip · Fevereiro 28, 2023, 9:36pm

@loic você pode dar uma olhada rápida nisso?

loic · Março 1, 2023, 8:07am

Claro, darei uma olhada

loic · Março 1, 2023, 11:46am

Corrigido aqui:

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 · Março 2, 2023, 9:39am

Reconstruindo a postagem de @awesomerobot e todos os links do Google Maps agora estão funcionando

Fechando isto.

Tópico		Respostas	Visualizações
Google Maps Onebox Issue Support	4	2325	3 de Maio de 2019
Google Maps onebox issues Bug	15	3594	1 de Janeiro de 2016
Embedding google maps short urls is broken Feature	9	7032	22 de Março de 2015
Embedded google map stopped working Support	10	4733	30 de Setembro de 2020
Google maps link not embedding anymore Support	2	652	12 de Setembro de 2022

Links incorporados do Google Maps às vezes funcionam, às vezes não (onebox 404)

Tópicos relacionados