selivanov_pavel Sep 16 2016 at 01:03

Изменение в PHP 7, ломающее при обновлении с пятой версии некоторые сервера на Ubuntu

2 min

14K

*nix*Server Administration*

From sandbox

+35

Comments 27

youlose Sep 16 2016 at 07:19

Если php.ini был скопирован из пятой версии
При чём тут Ubuntu Xenial, если вы скопировали этот файл вручную?
На убунте пхп 7 имеет отдельные папки для конфигов и она создаёт там новые конфиги при установке.

+10

imgen Sep 16 2016 at 09:51

Именно тот случай, где сделали — то, чего нельзя было делать и изобрели к нему костыли. К тому же у xenial по умолчанию стоят ppa не от ondrej/php.

selivanov_pavel Sep 16 2016 at 12:44

К тому же у xenial по умолчанию стоят ppa не от ondrej/php.

Проблема воспроизводится и в официальных пакетых от убунты, и в пакетах из ondrej/php.

selivanov_pavel Sep 16 2016 at 12:42

При чём тут Ubuntu Xenial, если вы скопировали этот файл вручную?

У нас этот файл раскладывается ролью ansible, и при переделывании роли с php5 на php7 я его просто скопировал и проверил по документации, что все используемые директивы работают так же. Вне зависимости от используемого метода поставки настроек(ansible, puppet, dockerfile, AWS AMI, ...) — при обновлении проще скопировать старый файл, чем в новом вручную прописывать те же настройки. Даже если переписывать файл с нуля — кто-то по привычке поставит # для комментария, и всё молча поломается.

DjOnline Sep 16 2016 at 10:43

Обратная совместимость? Нет, не знаем.

UFO just landed and posted this here

DjOnline Sep 16 2016 at 11:27

Обратная совместимость не должна зависеть от версий совсем, особенно в enterprise языках, как сейчас позиционирует себя PHP.

Fedcomp Sep 16 2016 at 12:34

Мажорная версия имеет право сбрасывать костыли, иначе такой full of legacy никому будет не нужен в будующем.

-1

DjOnline Sep 16 2016 at 12:58

Не имеет права. Иначе получается ужасная ситуация, когда есть тонны старого работающего кода, но обновлять версию нельзя из-за проблем совместимости, но и продолжать использовать старые версии тоже нельзя из-за прекращения поддержки и дыр в безопасности.

edogs Sep 16 2016 at 13:18

С такой логикой новые deprecated функции тоже не имеют право на существование, ведь из-за их наличия нельзя обновится, а старую версию нельзя использовать из-за дыр.

Проблема возникает только в том случае, когда конфиг файл накатили от несовместимой версии. Ну так не накатывайте, перенести настройки или проверьте его на совместимость. Вот и всё решение.

symbix Sep 16 2016 at 13:20

О том, что # — deprecated, предупреждало в startup errors года 3-4.

Вас послушать — так и register globals с magic quotes надо вечно тянуть было.

>обновлять версию нельзя из-за проблем совместимости
sed -i 's/^#/;/' php.ini
Ужасная проблема совместимости. Исправить уйдут человекогоды.

symbix Sep 16 2016 at 10:45

Это ж каким ССЗБ надо быть, чтобы обновить мажорную версию прямо на продакшене без тестирования.

selivanov_pavel Sep 16 2016 at 12:34

Так в том и весь прикол, что при тестировании всё работает, только со значениями по-умолчанию. Если не используются загрузки больших файлов и другие явно ломающиеся вещи, то деградацию можно не заметить: странички открываются, все функции приложения работают, 500 не выпадает. А то, что оно начало отдавать X-Powered-By:, больше не органичивает доступные PHP функции и на ошибки отдаёт пользователю стектрейс — с первого взгляда не видно.

symbix Sep 16 2016 at 13:14

У вас тестирование сводится к «странички открываются»? Acceptance-тесты, CI — не?

>больше не органичивает доступные PHP функции
а это вообще зачем, кроме шаред-хостинга?

edogs Sep 16 2016 at 13:21

Удивительно видеть в одном сообщении претензию к отсутствию полноценного тестирования и удивление по поводу ограничивания функций.

Вы наверное на сервере из под рута все время работаете? Исходя из логики «а зачем не из под рута, ведь доступ только у меня»:)

Дыры могут быть в любом сайте, отключение некоторых функций пхп это банально безопаснее, т.к. по крайней мере Вы можете быть уверены, что даже если Ваш пхп сайт ломанут, то хотя бы в консоли не смогут чего-нибудь выполнить через пхп.

symbix Sep 16 2016 at 22:15

Достаточно нормально настроить ограничения средствами операционной системы. Это и понадежнее будет.

edogs Sep 17 2016 at 01:50

Сознательно отказаться от дополнительного слоя безопасности? Ради чего?
Или Вы не в курсе что ОС тоже имеют уязвимости?
Trust noone:)

Вот, кстати, ничего так пример обхода ограничений https://habrahabr.ru/company/pentestit/blog/227497/

symbix Sep 17 2016 at 19:17

Отключение функций — это иллюзия безопасности, в том и дело.
Если совмещать, это другой вопрос.

edogs Sep 17 2016 at 22:08

Возможно Вам стоит почитать больше про отключение функций.
Отключение функций это повышение безопасности, а не иллюзия, т.к. доступ из консоли имеет больше возможностей и прав, чем доступ непосредственно их пхп.
А ограничение доступа это всегда хорошо, даже если не совмещать.
Да и пассаж про совмещать странный, для Вас не самоочевидно что подходы к ограничению надо совмещать?

selivanov_pavel Sep 16 2016 at 13:27

У вас тестирование сводится к «странички открываются»? Acceptance-тесты, CI — не?

Оно прошло тестирование, подробности — NDA. Разумеется, теперь мы добавили дополнительные проверки.

а это вообще зачем, кроме шаред-хостинга?

Немного уменьшаем поверхность атаки на случай уязвимостей в коде. Не факт что сильно поможет, но и не помешает

youlose Sep 16 2016 at 13:02

Вообще это похоже на проблему php-fpm, это же он стартует с неправильным конфигом?
Лучше им напишите репорт.

selivanov_pavel Sep 16 2016 at 14:02

По заявкам телезрителей: https://bugs.php.net/bug.php?id=73099

mxms Sep 16 2016 at 14:14

Не «сервера», а «серверы» будет по-русски.

-1

selivanov_pavel Sep 16 2016 at 14:33

https://ru.wiktionary.org/wiki/сервер:

В разговорной речи встречается также вариант склонения по схеме 1c(1) (мн. ч. сервера́, серверо́в, сервера́м, сервера́ми, сервера́х).

На мой вкус сервера звучит приятнее, но ваш вариант более академически привильный.

mxms Sep 16 2016 at 16:34

Во-первых, здесь не разговорная речь. Во-вторых, в разговорной речи, например, говорят «ихний» и «кушайте», что автоматически не делает эти слова часть> грамотного изъяснения на русском. И, наконец, в-третьих — вы, конечно, вольны делать что хотите, но окружающие будут судить о вашем образовании и интеллекте, в первую очередь, по степени владения языком.

-6

VolCh Sep 16 2016 at 20:56

Здесь как раз разговорная речь, а не литературная.

borNfree Sep 16 2016 at 21:38

по степени владения языком

надеюсь вы про PHP ;)

Show the best of all time