search

web-standards-ru / wsd.events

Web Standards Days
http://wsd.events
42 stars 29 forks source link

Исправляет политику CSP для API Яндекс.Карт #164

Closed alexbaumgertner closed 6 years ago

alexbaumgertner commented 6 years ago

Fix https://github.com/web-standards-ru/web-standards-days/issues/158

pepelsbey commented 6 years ago

Скажи, если я могу чем-то помочь, чтобы продвинуть этот пулреквест :) Потестить может чего.

alexbaumgertner commented 6 years ago

@pepelsbey я немного отвлекся, спасибо за напоминание про этот PR.

Помочь можно — потестировать этот PR и дать замечания.

pepelsbey commented 6 years ago

Задеплоил на wsd.events :) Ошибок не видно.

alexbaumgertner commented 6 years ago

@pepelsbey смотрю в Safari 12.0 (13606.2.11) — ошибок нет.

alexbaumgertner commented 6 years ago

Chrome 69.0.3497.100 (Official Build) (64-bit) тоже ок.

alexbaumgertner commented 6 years ago

@pepelsbey Можно наверное дать кому-нибудь третьему потестировать и если все ок — вливать :)

pepelsbey commented 6 years ago

@h4 глянешь? )

h4 commented 6 years ago

Что-то меня не очень радует количество заплаток, которые нужно добавить в разные места. А если потом ребята из Карт поменяют стратегию деплоя  — снова всё переделывать? В общем, я бы пошёл в Я.Карты и спросил у них самих — как сделать правильнее и пуленепробиваемее.

alexbaumgertner commented 6 years ago

@h4 тк этот pr чинит баг с показом карты в safari, предложу влить как есть, а в задаче добавить пункт про выяснение у Я.Карт более правильного пути.

alexbaumgertner commented 6 years ago

Проверил на страницах

— в последних Chrome и Safari ошибок нет. В Safari стала открываться карта на https://wsd.events/2018/08/18/.

pepelsbey commented 6 years ago

@alexbaumgertner спасибо! Кстати, если захочется сделать второй подход, то есть новое интересное расширение.