Встроенные ссылки Google Maps иногда работают, а иногда нет (onebox 404)

Canapin · 02.Ноябрь.2022 11:57:10

У меня наблюдается странное поведение с ссылками на Google Maps.

Иногда они корректно встраиваются, а иногда запрос на получение onebox возвращает 404. Мне не удалось воспроизвести эту проблему стабильно, но она возникает очень часто. Просто попробуйте вставить ссылки, переместить камеру или изменить область отображения в Google Maps, повторить попытку и так далее.

Сначала я подумал о лимите запросов (rate limit), но это не так. Ссылки, которые работают, и те, которые не работают, одинаковы как здесь, так и на моем форуме. Примеры:

У кого-нибудь есть идеи, откуда может исходить эта проблема? Со стороны Google? Со стороны Discourse? Стоит ли перенести это в Contribute > Bug?

awesomerobot · 21.Февраль.2023 21:07:11

Это как-то странно, я просто заметил закономерность… но, думаю, это связано с уровнем масштабирования в ссылке?

15.5z

15z

16.75z

17z

Совершенно другое местоположение…

15.38z

15z

Мне кажется, это проблема движка одностраничных карточек Google Maps на нашей стороне?

Falco · 21.Февраль.2023 22:02:46

Отличная работа, детектив @awesomerobot!

Похоже, наше регулярное выражение для уровня масштабирования не допускает точки здесь:

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

Это довольно легко исправить.

Canapin · 22.Февраль.2023 00:01:13

Какой же ловкий момент

Спасибо, что разобрались!

zogstrip · 28.Февраль.2023 21:36:29

@loic, можешь быстро взглянуть на это?

loic · 01.Март.2023 08:07:55

Конечно, я посмотрю

loic · 01.Март.2023 11:46:06

Исправлено здесь:

github.com/discourse/discourse

FIX: Allow floats for zoom level in Google Maps onebox (#20497)

main ← loic-fix-google-map-onebox

merged 11:45AM - 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 · 02.Март.2023 09:39:03

Восстановление поста @awesomerobot и все ссылки на Google Maps теперь работают

Закрываю.

Тема		Ответов	Просм.
Google Maps Onebox Issue Support	4	2325	03.05.2019
Google Maps onebox issues Bug	15	3594	01.01.2016
Embedding google maps short urls is broken Feature	9	6987	22.03.2015
Embedded google map stopped working Support	10	4721	30.09.2020
Google maps link not embedding anymore Support	2	650	12.09.2022

Встроенные ссылки Google Maps иногда работают, а иногда нет (onebox 404)

Связанные темы