VladimirIvanin / insales-uploader

Актуальная версия тут -
https://github.com/insales/insales-uploader
MIT License
9 stars 2 forks source link

404 на ссылках ассетов с доменом сайта #10

Closed alex-rd closed 6 years ago

alex-rd commented 6 years ago

Insales со вчерашнего дня отдает 404 при ссылках с доменом сайта. Соответственно скачивание темы на локальный компьютер невозможна.

VladimirIvanin commented 6 years ago

Добрый день!

Сегодня проверял скачивание, работает.

Видимо cdn был временно недоступен

alex-rd commented 6 years ago

Сообщение об "insales со вчерашнего дня отдает 404 при ссылках с доменом сайта." Я немного неправильно сформулировал. О том, что так теперь будет прислала тех.поддержка Insales, часть сообщения:

"Срочная новость! С сегодняшнего дня мы начнём отдавать ошибки 404 при запросе файлов темы через домен магазина. "

И ассеты не скачиваются (идет 404 ошибка). Любые .liquid файлы скачиваются, остальные нет

VladimirIvanin commented 6 years ago

Добавил фикс на урлы в 0.27.2

alex-rd commented 6 years ago

домен assets.insales.ru, будет в дальнейшем рабочим для тем, у которых файлы лежат на assets3.insales.ru (или assets и assets3 это просто сервера cdn и запрос можно делать к любому)?

VladimirIvanin commented 6 years ago

Сделал запрос, чтоб в ответе апи был домен ассета. Фикс который я сделал ненадежен, домены могут меняться, но пока хоть так( Спасибо за обратную связь!

alex-rd commented 6 years ago

Может вынести в настройки и адрес сервера, на котором лежат файлы (если не определен, то подставлять что-то по-умолчанию). Адрес можно всегда посмотреть на сайте, который в работе. (в качестве временной меры поможет)

VladimirIvanin commented 6 years ago

image

Добавил настройку в v0.27.3