ALT Linux Bugzilla – Full Text Bug Listing
| Summary: | Сделать поле формы "Новое название схемы" обязательным к заполнению. | ||
|---|---|---|---|
| Product: | Branch p8 | Reporter: | ashen <lex.shen> |
| Component: | alterator-openldap | Assignee: | Andrey Cherepanov <cas> |
| Status: | NEW --- | QA Contact: | qa-p8 <qa-p8> |
| Severity: | normal | ||
| Priority: | P3 | CC: | boyarsh, cas, grenka, klark.devel, manowar |
| Version: | не указана | ||
| Hardware: | all | ||
| OS: | Linux | ||
|
Description
ashen
2018-08-08 14:38:28 MSK
Перевешиваю на p8. Угу. Не посмотрел, что на сизифе openldap не стартует (у меня) с такой руганью: авг 08 19:07:19 ashennb.profpab.com slapd[21679]: looking for plugins in '/usr/lib64/sasl2-3', failed to open directory, error: No such file or directory авг 08 19:07:19 ashennb.profpab.com slapd[21679]: could not open config file "/etc/openldap/slapd.conf": Permission denied (13) Бага? (В ответ на комментарий №2) > Бага? Вряд ли. Скорее кривизна конфигурации. Сегодня как раз на Сизифе проверял ALD с alterator-openldap. Так вот там этой баги #35236 нет и такого поведения не наблюдается, но там вообще никакие файлы не загружаются, т.е. баг совсем другой. (В ответ на комментарий №3) > (В ответ на комментарий №2) > > Бага? > > Вряд ли. Скорее кривизна конфигурации. Вполне возможно. Пакеты ставил из сизифа, конфиги ручками не трогал > Сегодня как раз на Сизифе проверял ALD с > alterator-openldap. Так вот там этой баги #35236 нет и такого поведения не > наблюдается, но там вообще никакие файлы не загружаются, т.е. баг совсем > другой. Есть такое. Файлы схем не загружаются. А раз схемы не грузятся, то и наличие/отсутствие #35236 проверить нельзя. 09.08.2018 03:14, Leonid Krivoshein пишет: > А можете проверить, исправляет задание проблему и не вносит регрессий? > > apt-repo test 211195 На счёт регрессии не скажу, не заметил пока. Схема добавляется, но имя такое ... мягко скажем непонятное ... :) Имя файла схемы: MozillaLDAPAddressBook.Schema Имя схемы не задано в поле "Имя новой схемы:" Схема добавилась с таким именем: schema_filel4KDyt /etc/openldap/schema/schema_filel4KDyt.schema Считаю, что: 1) Если поле "Имя новой схемы:" пусто добавление должно блокировать, о чём пользователь должен быть извещён; 2) Дефолтное имя схемы следцет брать из имени файла схемы отбросив ".schema", а там уж как пользователь её обзовёт его дело. > 2) Дефолтное имя схемы следцет брать из имени файла схемы отбросив ".schema", а
> там уж как пользователь её обзовёт его дело.
Дефолтное имя не просто формировать из имени файла, а вставлять в поле "Имя новой схемы:", с возможностью его редактирования.
(В ответ на комментарий №5) > Считаю, что: > 1) Если поле "Имя новой схемы:" пусто добавление должно блокировать, о чём > пользователь должен быть извещён; Стандартное (ожидаемое) поведение для веб-форм: загрузка файла с тем именем, что и на хосте клиента, как опция: возможность его тут же переименовать. По замыслу авторов alterator-openldap, данное поле было изначально опционально. > 2) Дефолтное имя схемы следует брать из имени файла схемы отбросив ".schema", > а там уж как пользователь её обзовёт его дело. В коде сейчас именно так и сделано: http://git.altlinux.org/tasks/index/p8/eperm/211195/gears/100/git?p=git;a=commitdiff;h=79d275d3e85550e280c9c78eac57aa19e85c6dd9 Но для меня загадка, почему файл загружается с таким именем. Видимо это уже guile, надо спрашивать manowar@. (В ответ на комментарий №6) > Дефолтное имя не просто формировать из имени файла, а вставлять в поле "Имя > новой схемы:", с возможностью его редактирования. По-моему, проблема и решение на стороне фронт-энда: либо gule, либо ajax. Моя попытка исправить бэкэнд привела к не совсем ожидаемому результату, так что оставлю решение проблемы фронт-эндовцам. (В ответ на комментарий №8) > (В ответ на комментарий №6) > > Дефолтное имя не просто формировать из имени файла, а вставлять в поле "Имя > > новой схемы:", с возможностью его редактирования. > > По-моему, проблема и решение на стороне фронт-энда: либо gule, либо ajax. Моя > попытка исправить бэкэнд привела к не совсем ожидаемому результату, так что > оставлю решение проблемы фронт-эндовцам. Соглашусь. Но в бакенде нужно обязательно тоже сделать проверку на непустое значение. И слать write_error с пояснительным сообщением, если пустое. Бакенд должен защищать себя от некорректного использования. |
