andreyryabin / sprint.migration

Миграции для разработчиков (1С-Битрикс)
http://marketplace.1c-bitrix.ru/solutions/sprint.migration/
MIT License
173 stars 56 forks source link

Fixed example of console runner script #2

Closed webarchitect609 closed 7 years ago

webarchitect609 commented 7 years ago

Предлагаю внести исправление в документацию, где неверно дан пример файла для запуска из консоли.

Во-первых, нет защиты от запуска через HTTP. Файл должен запускаться только из консоли. Во-вторых, о каком $_SERVER['DOCUMENT_ROOT'] может идти речь при запуске из консоли? Нет там никакого $_SERVER.

Единственное, что я не тестил, это можно ли вообще обойтись без определения $_SERVER['DOCUMENT_ROOT']

andreyryabin commented 7 years ago

Спасибо за предложения Что касается защиты через http - она есть в файле https://github.com/andreyryabin/sprint.migration/blob/master/tools/migrate.php

if (!empty($_SERVER["HTTP_HOST"])){ die('console only'); }

а $_SERVER['DOCUMENT_ROOT'] - нужен битриксу, он должен быть определен