
Sergey
26.12.2017
09:04:30
да, в этом случае все будет хорошо

Roman
26.12.2017
09:04:52

Shmaltorhbooks
26.12.2017
09:04:53
ArticlesCollection как-то правильнее, имхо

Google

Sergey
26.12.2017
09:04:57
и это в целом будет весьма удобно.... возможно

Roman
26.12.2017
09:05:31

Sergey
26.12.2017
09:05:43
клиентский код будет завязан только на интерфейс а потому все будет ок

Roman
26.12.2017
09:06:17

Sergey
26.12.2017
09:06:20
если контракты этих двух имплементаций будут соблюдены

Roman
26.12.2017
09:09:57
Короче, можно пробовать.
На первый взгляд должно получиться красиво и лаконично.
Только интересно, что получится если заюзать matching(SomeSpec $spec). Хотя вряд ли понадобится

Sergey
26.12.2017
09:12:45
НО! текущий вариант будет говном, либо надо геттеры
ибо вариант с DQL будет по базе искать а вот ArrayCollection будет уже через филды опрашивать
НО! это все легко и просто меняется своим визитором на основе дефолтного, я так делал - удобненько)

Google

Sergey
26.12.2017
09:14:29
из преимуществ - у тебя не будет особо разницы между коллекциями сущностей и репозиториями, можно будет выстраивать взаимоотношения более естественным образом (как если бы у тебя всегда была какая-то сущность)

Иван
26.12.2017
09:14:34

Sergey
26.12.2017
09:14:46
ну и еще - если в 3.0 таки запилят возможность кастомные коллекции делать - будет еще прикольнее
все логи контейнера по идее должны пробрасываться в stdout/stderr контейнера
и доступны в docker logs

Shmaltorhbooks
26.12.2017
09:15:33
в докер вроде сислог может быть перенаправлен в stdout и еще можно отдельно сконфигурить куда он будет писаться
а для начала - попробуй в крон прописать что-то своё, простое

Sergey
26.12.2017
09:15:48
проверить есть ли крон или нет ты можешь зайдя в контейнер и руками попробовать запустить крон

Shmaltorhbooks
26.12.2017
09:15:52
типа date » /tmp/foo.txt
на каждую минуту
если файл создастся - крон шуршит
тогда копать в сторону самой аппликухи

Иван
26.12.2017
09:16:15

Shmaltorhbooks
26.12.2017
09:16:50
какая ось в контейнере?

Sergey
26.12.2017
09:17:08
как руками запустить крон?
ты не поверишь, но crontab -l. Почитай доки по ларадок. Я думаю они не настолько убогие так что крон должен запускаться отдельно отдельным энтрипоинтом

Иван
26.12.2017
09:17:11

Sergey
26.12.2017
09:17:16
это контейнер

Google

Roman
26.12.2017
09:17:32

Shmaltorhbooks
26.12.2017
09:17:43
команды могут быть разными

Sergey
26.12.2017
09:18:03

Roman
26.12.2017
09:18:50

Sergey
26.12.2017
09:19:59
точку входа в контейнер найди, там должно быть что-то типа cron -f
что бы крон работал

Shmaltorhbooks
26.12.2017
09:20:45
и вообще - сам крон работает?
то понятно, что это контейнер ларовский и всё такое
но работает или нет?
$ ps ax | grep cron
что пишет?
$ service --status-all

Иван
26.12.2017
09:22:01

Shmaltorhbooks
26.12.2017
09:22:51
$ crontab -l

Иван
26.12.2017
09:23:14

Shmaltorhbooks
26.12.2017
09:26:52
su laradock
crontab -e
добавь что-то и
crontab -l

Иван
26.12.2017
09:26:54
странно, в файле laradock нет моей строки...только ларавская. Попробую перебилдить контейнеры, проверю, отпишусь

Shmaltorhbooks
26.12.2017
09:27:19
може у тебя файло с хоста при билде не скопировалось или кроном не подхватились конфиги

Google

Иван
26.12.2017
09:28:03
Спасибо всем!

Shmaltorhbooks
26.12.2017
09:52:32
Таки файло её скопировалось?
не*

Alexander
26.12.2017
13:24:58
Запускать стороннее приложение из процесса php-fpm - это плохо для памяти, производительности и безопасности? В частности говорю про wkhtmltopdf.org

Иван
26.12.2017
13:25:27

Admin
ERROR: S client not available

Sergey
26.12.2017
13:27:38
через какой proc_open в целом норм
весь вопрос в санитайзинге аргументов
если ты данные запроса не юзаешь для этого - то почему бы и нет
ну или можно поднять маленький RPC сервачек который будет на вход получать html и на выход давать тебе путь к pdf (мне так удобнее))

Alexander
26.12.2017
13:30:08
Поясни плизз про "Санитайзинг аргументов"? То есть если я вызываю процесс с аргументами то это плохо?

Sergey
26.12.2017
13:31:55
`
exec("${GET['cmd']}"); думаю не очень хорошо)

Alexander
26.12.2017
13:32:14
ААААааа! понял. Ну конечно нет такого.

Sergey
26.12.2017
13:32:19
а если ты контролируешь и генеришь аргументы и туда нельзя подсунуть rm -rf то почему бы нет

Alexander
26.12.2017
13:33:52
Да понял. Точно нельзя. Комманда проверяется. Перед тем как сделать exec. Я просто код ревью не прошел с таким решением. Сказали что это "плохо для памяти, производительности и безопасности"
Мне легче переделать, чем спорить и что то доказывать.

Google

Alexander
26.12.2017
13:34:20
Спасибо

Dmitry
26.12.2017
13:35:58
от контекста зависит в общем... единственный реальный минус - процесс тяжелый, легко скушать весь пул пхп под нагрузкой, но не всегда это актуально

Alexander
26.12.2017
13:39:11
Это же надо как то замерить что процесс тяжелый? Генерация PDF занимает доли секунд. Не ясно на сколько нагружен сервис. Скольким людям потребуется одновременно сохранять в PDF, то что они в данный момент видят на странице.

Sergey
26.12.2017
13:41:58
в любом случае ты упрешься в максимальное количество воркеров fpm
и от этого количества можно уже посчитать расход ресурсов
ну либо очереди) тогда все красиво но сложнее)

Dmitry
26.12.2017
13:44:38
я бы просто смотрел в сторону отдельного пула для этого, и заруливать туда на уровне веб-сервера... что бы были свои лимиты на это дело, и не храл пул для показа страничек

Alexander
26.12.2017
13:47:44
Я правильно понял что нужно проводить тесты на нагрузку, для того чтобы понять подойдет такое решение генерации PDF или нет? Елси я беру к примеру mPDF для генерации файла, у меня ведь также отъедаются ресурсы на генерацию файла и того же RAM ?
Поисковики не пройдут форму авторизации пользователя. Проект внутрений и закрыт от поисковиков.

Dmitry
26.12.2017
13:49:36
ну я и говорю - вопрос контекста...
как хоть тебе предложили переделать?

Alexander
26.12.2017
13:52:31
Сказали предложить другой вариант. Конкретных указаний не было. Кивнули в сторону mPDF. Но у меня стоит mbstring func_overload=2. С ним mPDF падает по excpetion.

Bohdan
26.12.2017
13:54:31
единственный нюанс - она по дефолту в докере поставляется

Dmitry
26.12.2017
13:54:57
мдя, что-то не уверен я, что purephp будет производительнее, чем fork+exec программы на с ;)

Alexander
26.12.2017
13:57:05
ну я о том же. Не совсем понял, чем не нравится такой подход.( Потом после того как переделаю, задам вопрос.

Dmitry
26.12.2017
13:59:16
ну в общем exec многих раздражает... но аргументировать не всегда могут
впрочем, func_overload меня тоже раздражает ;)

Alexander
26.12.2017
14:02:53
https://t.me/prophp7/85174
ну вот поэтому я думаю раздражает.
В новых версиях не будет func_overload. Забудем как страшный сон)