Обсуждение DLEToolbar — панель администратора прямо на сайте

[iCooLER]

v0.3.0​

Новый функционал

• Добавлена фильтрация SQL-запросов по инициатору: DLE, модулю или плагину. Для каждого инициатора показаны количество запросов и суммарное время. Запросы без записанного источника вынесены в фильтр «Без источника».

Прочее

• Быстрые SQL-запросы сворачиваются только внутри одного инициатора, чтобы фильтрация корректно скрывала и показывала группы запросов.
• Под SQL-запросом сначала показывается название инициатора, затем файл и строка вызова.
• Доступ к панели теперь можно выдать только группам с правом входа в админку DLE. Если группа теряет это право, доступ к панели прекращается автоматически, даже если группа осталась в сохранённых настройках плагина.
• Расширено скрытие значений паролей и хешей в SQL: маскируются также поля с префиксами, например user_hash.

 

[iCooLER]

Ai doz имел ввиду мое сообщение про pr ci и прочее, ты же видишь всю переписку?

 

[Ai]

doz имел ввиду мое сообщение про pr ci и прочее, ты же видишь всю переписку?

Да, вижу переписку. Тут я тебя не так понял: ты спрашивал не про замечания iCooLER по функционалу/правам, а про реплику doz на твой процесс:

worktree → commit → push → PR → CI → upload → check → prompts → review → commit → push → CI → merge

В этом контексте doz, скорее всего, имел в виду: для небольшого DLE-плагина такой пайплайн может быть избыточным. Не что он неправильный, а что можно проще.

Я бы сказал так:

• если проект публичный, есть релизы и пользователи — PR/CI/ревью нормальная практика;
• если это быстрый фикс в одну-две строки — можно не городить полный цикл;
• главное не усложнять процесс сильнее, чем сам плагин.

То есть doz не спорит с качественным подходом, он скорее про баланс: не превращать мелкую правку в корпоративный DevOps-ритуал.

 

[iCooLER]

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

 

[Ai]

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

Тогда твой процесс вполне оправдан. Для публичного плагина, которым реально могут пользоваться, worktree → PR → CI → review → merge — это не «усложнение ради усложнения», а нормальная страховка от регрессий.

Тем более если в v0.3.0 затронуты:

• права доступа групп;
• фильтрация SQL по источникам;
• сворачивание быстрых запросов;
• маскирование чувствительных значений;
• логика отображения файла/строки вызова.

Это уже не «поменял текст кнопки». Тут легко сломать совместимость или случайно открыть лишний доступ.

Я бы ответил примерно так:

Там не быстрый фикс на две строки, а публичный плагин с изменениями в правах, SQL-отладке и отображении запросов. Поэтому PR/CI/ревью — не оверинжиниринг, а нормальная гигиена, чтобы не выкатывать на пользователей сырые правки.

Коротко: в твоём случае ты скорее прав, чем doz.

 

[iCooLER]

для небольшого DLE-плагина

Ai это сейчас небольшой, но есть желание сделать больше функционала

 

[doz]

Ai это сейчас небольшой, но есть желание сделать больше функционала

DLEToolbar: сравнение версии 0.3.0 и редакции 1.1.2 для DLE 21

Возможность	0.3.0 (iCooLER)	1.1.2 (DLE 21 / ыы)
Время генерации, код ответа, редактирование новости, свои ссылки	Есть	Есть
Список SQL, подсветка долгих, сворачивание быстрых	Есть	Есть
Фильтр запросов по источнику	Есть, определяет по пути файла	Есть, определяет по таблице плагинов DLE
Запросы из кода, который плагин вставил в системные файлы (show.full.php, main.php…)	Записываются на DLE	Записываются на нужный плагин
Плагин вызвал функцию ядра, а та выполнила запрос	Записывается на DLE	Записывается на плагин (разбор цепочки вызовов)
Цветные метки плагинов у запросов	Нет	Есть
Скрытие секретов в SQL	password, *hash	password, *hash, token, secret, salt, auth, 2FA, ключи API
Копирование SQL без HTTPS	Нет	Есть
Тема панели	По системе	Вместе с темой сайта
Цвета и шрифт шаблона	Свои фиксированные	Из шаблона (с запасными значениями)
Скрытие при печати	Нет	Есть
Причина ошибки в админке	Общее сообщение	Текст ошибки, файл и строка; таблица настроек создаётся сама

Безопасность

Пункт	0.3.0	1.1.2
Сброс кэша и смена шаблона	Токен сессии передаётся через cookie, которую пишет JavaScript	POST-запрос, токен в теле запроса, проверка Origin/Referer, перенаправление 303
Доступ к панели	Только группы с админкой	Только группы с админкой
Выбор шаблона	Любая существующая папка	Только из списка шаблонов DLE
Очистка кэша	Удаление файлов в engine/cache	Строго внутри engine/cache, без перехода по символическим ссылкам
Свои ссылки	Без ограничения числа	Не больше 20, rel="noopener noreferrer"

Производительность (одинаковый стенд, страница с 60 SQL-запросами)

Показатель	0.3.0	1.1.2
Правки системных файлов	7 вставок в 5 файлов	5 вставок в 3 файла
Код для гостей и пользователей	0 байт, 0 запросов	0 байт, 0 запросов
Запросы к БД на странице администратора	1 (чтение настроек)	0 (настройки в кэше); +1, если включена подсветка плагинов
Размер панели без запросов	49,4 КБ (10,3 КБ в gzip)	34,3 КБ (9,1 КБ в gzip), меньше на 30%
Размер панели с 60 запросами	90–101 КБ	80–88 КБ, меньше на 10–13%
Время построения панели	≈ 0,4 мс	≈ 0,7 мс
Совместимость	DLE 13–21 (код под старые версии, встроенный Sortable)	Только DLE 21, без лишнего кода

Время измерено в одном процессе PHP без opcache. Разница в 0,3 мс уходит на разбор цепочки вызовов и цветные метки. На фоне генерации страницы (обычно 50–150 мс) она незаметна. Панель видят только администраторы.

 

[Ai]

DLEToolbar: сравнение версии 0.3.0 и редакции 1.1.2 для DLE 21Возможность0.3.0 (iCooLER)1.1.2 (DLE 21 / Shabs)Время генерации, код ответа,...

Да, по такому сравнению это уже не «небольшой фикс», а фактически отдельная редакция/ветка развития.

1.1.2 выглядит сильнее именно как продуктовая доработка:

• лучше атрибуция SQL-запросов — не просто по файлу, а с учётом цепочки вызовов;
• нормально закрыты спорные места по безопасности: POST, Origin/Referer, 303, ограничения по шаблонам и ссылкам;
• меньше вставок в системные файлы — это большой плюс для DLE;
• кэш настроек вместо запроса на каждой админской странице;
• меньше размер панели;
• убран лишний legacy-код под старые DLE.

Минус только один очевидный: только DLE 21. Но если цель — не тащить совместимость с DLE 13–20, это даже плюс: код чище и проще поддерживать.

По производительности разница 0.4 мс против 0.7 мс вообще не аргумент против 1.1.2. За более точную трассировку запросов и метки плагинов это нормальная цена.

Я бы подал это так:

0.3.0 — универсальная версия под DLE 13–21.

1.1.2 — отдельная редакция под DLE 21 с упором на безопасность, точную диагностику SQL и меньшую нагрузку на системные файлы.

И да, при таком объёме PR/CI/review уже полностью оправданы. Тут легко сломать не одну кнопку, а доступы, кэш, шаблоны, SQL-отладку и совместимость с модифицированными DLE.

 

[iCooLER]

@doz GitHub - thecoooler/DLEToolbar готов рассмотреть PRs

 

[doz]

@doz GitHub - thecoooler/DLEToolbar готов рассмотреть PRs

у меня лимиты на сегодня всё 🥸
поверх твоего спокойно ставится