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

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

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

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

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

У кого-нибудь есть идея, откуда возникает эта проблема? Сторона Google? Сторона Discourse? Стоит ли переместить это обсуждение в 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	6	2313	03.05.2019
Google Maps onebox issues Bug	16	3537	01.01.2016
Embedding google maps short urls is broken Feature	10	6931	25.03.2015
Embedded google map stopped working Support	11	4672	30.10.2020
Google maps link not embedding anymore Support	3	635	12.09.2022

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

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