Bug 35236 - Сделать поле формы "Новое название схемы" обязательным к заполнению.
Summary: Сделать поле формы "Новое название схемы" обязательным к заполнению.
Status: NEW
Alias: None
Product: Branch p8
Classification: Distributions
Component: alterator-openldap (show other bugs)
Version: не указана
Hardware: all Linux
: P3 normal
Assignee: Andrey Cherepanov
QA Contact: qa-p8@altlinux.org
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-08-08 14:38 MSK by ashen
Modified: 2018-08-10 20:50 MSK (History)
5 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description ashen 2018-08-08 14:38:28 MSK
Служба каталогов LDAP -> управление схемами -> выберите файл -> <выбор файла схемы> -> добавить.

Если не заполнено поле "Новое название схемы", то, как имя добавляемой схемы пусто, так и файл схемы именуется как ".schema":

/etc/openldap/schema/.schema.

См. скриншоты.

https://yadi.sk/i/xvFGKRJy3a2aRf

https://yadi.sk/i/JQxKoE1N3a2aYG

В /etc/openldap/slapd.conf появляется такая запись:

#include                /etc/openldap/schema/.schema

И /etc/openldap/schema имеем:

/etc/openldap/schema/.schema

Правка /etc/openldap/slapd.conf исправляет ситуацию.
Comment 1 Grigory Ustinov 2018-08-08 18:50:38 MSK
Перевешиваю на p8.
Comment 2 ashen 2018-08-08 19:19:57 MSK
Угу. Не посмотрел, что на сизифе 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)

Бага?
Comment 3 Leonid Krivoshein 2018-08-09 00:54:12 MSK
(В ответ на комментарий №2)
> Бага?

Вряд ли. Скорее кривизна конфигурации. Сегодня как раз на Сизифе проверял ALD с alterator-openldap. Так вот там этой баги #35236 нет и такого поведения не наблюдается, но там вообще никакие файлы не загружаются, т.е. баг совсем другой.
Comment 4 ashen 2018-08-09 07:41:36 MSK
(В ответ на комментарий №3)
> (В ответ на комментарий №2)
> > Бага?
> 
> Вряд ли. Скорее кривизна конфигурации.

Вполне возможно. Пакеты ставил из сизифа, конфиги ручками не трогал

> Сегодня как раз на Сизифе проверял ALD с
> alterator-openldap. Так вот там этой баги #35236 нет и такого поведения не
> наблюдается, но там вообще никакие файлы не загружаются, т.е. баг совсем
> другой.

Есть такое. Файлы схем не загружаются. А раз схемы не грузятся, то и наличие/отсутствие #35236 проверить нельзя.
Comment 5 ashen 2018-08-09 09:46:56 MSK
09.08.2018 03:14, Leonid Krivoshein пишет:
> А можете проверить, исправляет задание проблему и не вносит регрессий?
>
> apt-repo test 211195 

На счёт регрессии не скажу, не заметил пока.
Схема добавляется, но имя такое ... мягко скажем непонятное ... :)
Имя файла схемы:

MozillaLDAPAddressBook.Schema

Имя схемы не задано в поле "Имя новой схемы:"
Схема добавилась с таким именем:


schema_filel4KDyt	/etc/openldap/schema/schema_filel4KDyt.schema


Считаю, что:
1) Если поле "Имя новой схемы:" пусто добавление должно блокировать, о чём пользователь должен быть извещён;
2) Дефолтное имя схемы следцет брать из имени файла схемы отбросив ".schema", а там уж как пользователь её обзовёт его дело.
Comment 6 ashen 2018-08-09 09:49:12 MSK
> 2) Дефолтное имя схемы следцет брать из имени файла схемы отбросив ".schema", а
> там уж как пользователь её обзовёт его дело.

Дефолтное имя не просто формировать из имени файла, а вставлять в поле "Имя новой схемы:", с возможностью его редактирования.
Comment 7 Leonid Krivoshein 2018-08-09 11:47:30 MSK
(В ответ на комментарий №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@.
Comment 8 Leonid Krivoshein 2018-08-10 19:31:33 MSK
(В ответ на комментарий №6)
> Дефолтное имя не просто формировать из имени файла, а вставлять в поле "Имя
> новой схемы:", с возможностью его редактирования.

По-моему, проблема и решение на стороне фронт-энда: либо gule, либо ajax. Моя попытка исправить бэкэнд привела к не совсем ожидаемому результату, так что оставлю решение проблемы фронт-эндовцам.
Comment 9 manowar@altlinux.org 2018-08-10 20:50:45 MSK
(В ответ на комментарий №8)
> (В ответ на комментарий №6)
> > Дефолтное имя не просто формировать из имени файла, а вставлять в поле "Имя
> > новой схемы:", с возможностью его редактирования.
> 
> По-моему, проблема и решение на стороне фронт-энда: либо gule, либо ajax. Моя
> попытка исправить бэкэнд привела к не совсем ожидаемому результату, так что
> оставлю решение проблемы фронт-эндовцам.

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