Bug 41391

Summary: Не добавляется фотография при создании пользователя
Product: Sisyphus Reporter: Vera Blagoveschenskaya <vercha>
Component: alterator-ldap-usersAssignee: manowar <manowar>
Status: CLOSED FIXED QA Contact: qa-sisyphus
Severity: normal    
Priority: P5 CC: boyarsh, cas, manowar, sem
Version: unstable   
Hardware: x86_64   
OS: Linux   
See Also: https://bugzilla.altlinux.org/show_bug.cgi?id=41389
Attachments:
Description Flags
screen none

Description Vera Blagoveschenskaya 2021-11-19 16:33:36 MSK
Created attachment 9950 [details]
screen

Тестовый стенд: 
dc p10 server x86_64
client p10 workstation x86_64

[root@dc ~]# rpm -qa | grep alterator-ldap
alterator-ldap-groups-0.6.7-alt1.x86_64
alterator-ldap-users-0.8.5-alt2.x86_64

[root@dc ~]# rpm -qa | grep netcmdplus
python3-module-netcmdplus-0.1.2-alt1.noarch
netcmdplus-0.1.2-alt1.noarch

[root@dc ~]# rpm -qa | grep samba
samba-client-4.14.8-alt1.x86_64
samba-winbind-4.14.8-alt1.x86_64
samba-dc-common-4.14.8-alt1.noarch
samba-common-4.14.8-alt1.noarch
admx-samba-4.14.8-alt1.noarch
samba-libs-4.14.8-alt1.x86_64
samba-common-tools-4.14.8-alt1.x86_64
samba-winbind-clients-4.14.8-alt1.x86_64
python3-module-samba-4.14.8-alt1.x86_64
samba-common-client-4.14.8-alt1.noarch
samba-winbind-common-4.14.8-alt1.x86_64
samba-dc-libs-4.14.8-alt1.x86_64
samba-pidl-4.14.8-alt1.noarch
samba-dc-4.14.8-alt1.x86_64
samba-4.14.8-alt1.x86_64
samba-dc-client-4.14.8-alt1.x86_64
samba-common-libs-4.14.8-alt1.x86_64

1) Развернуть Samba домен, установить пакет netcmdplus, открыть web интерфейс альтератора. Открыть вкладку Пользователи, нажать Выбор источника, выбрать Samba ActiveDirectory и выбрать имя домена. Применить.
2) Создать нового пользователя.
3) Добавить фотографию -> Указать путь (добавляла jpg 60 kb) -> Установить -> Сохранить параметры

Результат: фото не сохранено

Ожидаемый результат: 
1) Фото добавлено
2) Возле поля для добавления фото отображаются допустимые параметры для фото (формат, размер)
Comment 1 manowar@altlinux.org 2021-12-03 15:21:15 MSK
Я пока не нашёл информации о том, что Samba AD/DC поддерживает фотографии пользователей. Поэтому могу предложить сообщение об ошибке. Также можно скрывать элементы формы, связанные с фотографией для режима ActiveDirectory.
Comment 2 manowar@altlinux.org 2021-12-06 20:44:10 MSK
Попробуй https://git.altlinux.org/tasks/291520/ .
Comment 3 Vera Blagoveschenskaya 2021-12-07 09:58:04 MSK
На чистом стенде с добавлением таска, проверить не получается:

# apt-repo add 291520
[root@dc ~]# apt-get update && apt-get dist-upgrade 
Получено: 1 http://git.altlinux.org repo/291520/x86_64 release [1672B]
Получено: 2 http://git.altlinux.org repo/291520/x86_64-i586 release [1642B]
Получено 3314B за 0s (134kB/s).
Получено: 1 http://git.altlinux.org repo/291520/x86_64/task pkglist [6204B]
Получено: 2 http://git.altlinux.org repo/291520/x86_64/task release [140B]
Получено: 3 http://git.altlinux.org repo/291520/x86_64-i586/task pkglist [2224B]
Получено: 4 http://git.altlinux.org repo/291520/x86_64-i586/task release [145B]
Получено 8713B за 0s (14,5kB/s).
Чтение списков пакетов... Завершено
Построение дерева зависимостей... Завершено
Чтение списков пакетов... Завершено
Построение дерева зависимостей... Завершено
Подсчет обновлений... Завершено
Следующие пакеты будут ОБНОВЛЕНЫ:
  alterator-usersource-functions
Следующие пакеты будут УДАЛЕНЫ:
  alt-domain-server alterator-ldap-users
Следующие пакеты будут СОХРАНЕНЫ:
  alterator-fbi libguile-vhttpd libvhttpd vhttpd-utils

Посмотрела на стенде, подготовленном для https://bugzilla.altlinux.org/41389, там фикс уже есть (но версии пакетов alterator-fbi libguile-vhttpd libvhttpd vhttpd-utils старые)
Фото добавить нельзя (нет поля).
Comment 4 manowar@altlinux.org 2021-12-07 13:49:42 MSK
> Следующие пакеты будут СОХРАНЕНЫ:
>  alterator-fbi libguile-vhttpd libvhttpd vhttpd-utils

Вот тут проблема, очевидно. Что-то сопротивляется обновлению libvhttpd.
Comment 5 manowar@altlinux.org 2021-12-07 14:40:28 MSK
А, всё просто — это потому, что задание для Сизифа.
Comment 6 manowar@altlinux.org 2021-12-07 14:56:14 MSK
Попробуйте задание для p10: https://git.altlinux.org/tasks/291571/ .
Comment 7 Vera Blagoveschenskaya 2021-12-07 15:12:05 MSK
(Ответ для manowar@altlinux.org на комментарий #6)
> Попробуйте задание для p10: https://git.altlinux.org/tasks/291571/ .

Подтверждаю, фикс есть. И заодно https://bugzilla.altlinux.org/41514 перестал воспроизводиться с таском.
Comment 8 manowar@altlinux.org 2021-12-07 15:56:48 MSK
(Ответ для Vera Blagoveschenskaya на комментарий #7)
> И заодно https://bugzilla.altlinux.org/41514
> перестал воспроизводиться с таском.

Не могёт этого быть. Там всё дело в фокусе. CSS я не трогал, потому что так слёту не знаю, какое свойство отвечает за цвет активного элемента при потере фокуса. А во вторых, брэндинги у нас вроде кто-то специально делает.
Comment 9 manowar@altlinux.org 2021-12-07 16:03:07 MSK
(Ответ для Vera Blagoveschenskaya на комментарий #7)
> (Ответ для manowar@altlinux.org на комментарий #6)
> > Попробуйте задание для p10: https://git.altlinux.org/tasks/291571/ .
> 
> Подтверждаю, фикс есть.

Тут ещё такой момент: я убрал некоторые рудименты (как мне показалось) в alterator-fbi, которые отвечают за выгрузку (upload) файла. Там фишка была в том, что вместе с файлом отправлялась и вся форма впридачу, все поля ввода. Теперь же upload делает только upload одного выбранного файла. И поэтому хорошо бы теперь проверить, что с новой версией alterator-fbi (из задания) не сломались другие модули alterator, в которых была выгрузка файла на сервер.

Мне кажется, их у нас немного. В принципе, я могу попробовать выяснить, какие именно это модули, по коду. Но может быть это уже где-то отмечено?
Comment 10 manowar@altlinux.org 2022-04-19 14:17:19 MSK
Давайте пока на Сизиф: там закроем, а потом уже в p10.
Comment 11 manowar@altlinux.org 2022-04-20 11:43:12 MSK
alterator-ldap-users 0.8.5-alt2 -> 0.8.6-alt2
 Tue Apr 19 2022 Paul Wolneykien <manowar@altlinux> 0.8.6-alt2
 - Switch to the branch's default version of Guile (guile-devel).
 Mon Dec 06 2021 Paul Wolneykien <manowar@altlinux> 0.8.6-alt1
 - Require alterator-fbi >= 5.49.3 for upload and UI fixes.
 - Hide the photo frame for AD (closes: 41391).
 - Fix current user selection in the user list.
 - Fix: Return error on trying to upload a photo for AD user.
 - Display the unset login shell as 'Default shell' (closes: 41389).