Кнопка "Зашифровать все пароли" не работает #243

Closed
opened 7 months ago by anshn · 15 comments
anshn commented 7 months ago
Collaborator

Необходимо проверить кнопку "Зашифровать все пароли" в окне дополнительных настроек.

  1. После ее нажатия ничего не происхожит визуально - нет никакого статуса, ничего. Непонятно работает ли.

  2. После того как установил новый алгоритм (в моем случае gost-yescrypt) - возвращаешься в это окно, нажимаешь на кнопку и в конфиге ничего не меняется. Все пароли как были заданы в yescrypt так и остаются в нем.

Необходимо проверить кнопку "Зашифровать все пароли" в окне дополнительных настроек. 1. После ее нажатия ничего не происхожит визуально - нет никакого статуса, ничего. Непонятно работает ли. 2. После того как установил новый алгоритм (в моем случае gost-yescrypt) - возвращаешься в это окно, нажимаешь на кнопку и в конфиге ничего не меняется. Все пароли как были заданы в yescrypt так и остаются в нем.
anshn added the ошибка label 7 months ago
YanTheKaller was assigned by anshn 7 months ago
Collaborator

"Зашифровать все пароли" шифрует только нешифрованные пароли

"Зашифровать все пароли" шифрует только нешифрованные пароли
anshn commented 7 months ago
Poster
Collaborator

Жесть бесполезная кнопка.

Значит только статус какой то добавить бы, что шифровать нечего и что шифрование выполнено успешно. Например предлагаю:

Шифрование паролей выполнено успешно.

В конфигурационном файле нет открытых паролей для шифрования.

Жесть бесполезная кнопка. Значит только статус какой то добавить бы, что шифровать нечего и что шифрование выполнено успешно. Например предлагаю: > Шифрование паролей выполнено успешно. > В конфигурационном файле нет открытых паролей для шифрования.
asmeron was assigned by anshn 7 months ago
Collaborator

В конфигурационном файле нет открытых паролей для шифрования.

Не получится, команда
/usr/lib/ublinux/functions globalconf_convert_pass_plain_to_hash
возвращает 0 (успешное выполнение) даже если незашифрованных паролей в конфиге не было. Судя по выводу, этот скрипт загружает все параметры с паролями без проверки хэш это или пароль, проверяет его только перед тем как зашифровать пароль и перезаписывает всё что загружал в конфиг. Проще говоря, если у нас 10 пользователей и только один из них без зашифрованного пароля, то скрипт загрузит всех десятерых, зашифрует пароль у одного и сохранит всех десятерых обратно

> > В конфигурационном файле нет открытых паролей для шифрования. Не получится, команда /usr/lib/ublinux/functions globalconf_convert_pass_plain_to_hash возвращает 0 (успешное выполнение) даже если незашифрованных паролей в конфиге не было. Судя по выводу, этот скрипт загружает все параметры с паролями без проверки хэш это или пароль, проверяет его только перед тем как зашифровать пароль и перезаписывает всё что загружал в конфиг. Проще говоря, если у нас 10 пользователей и только один из них без зашифрованного пароля, то скрипт загрузит всех десятерых, зашифрует пароль у одного и сохранит всех десятерых обратно
anshn commented 6 months ago
Poster
Collaborator

image

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

![image](/attachments/bedefbdb-6eec-4824-a2d5-d927446d5a4d) Предупреждение о том что шифрование было выполнено все равно считаю необходимым. Непонятно в какой момент происходит шифрование, сейчас или после кнопки сохранить, поэтому нужно хотя бы уведомление о том что что-то было сделано
Owner

"Зашифровать все пароли"
Добавить правильно описание, что зашифровать все пароли хранимые в не зашифрованном виде в конфигурации.

"Зашифровать все пароли" Добавить правильно описание, что зашифровать все пароли хранимые в не зашифрованном виде в конфигурации.
Owner

В конфигурационном файле нет открытых паролей для шифрования.

Не получится, команда
/usr/lib/ublinux/functions globalconf_convert_pass_plain_to_hash
возвращает 0 (успешное выполнение) даже если незашифрованных паролей в конфиге не было. Судя по выводу, этот скрипт загружает все параметры с паролями без проверки хэш это или пароль, проверяет его только перед тем как зашифровать пароль и перезаписывает всё что загружал в конфиг. Проще говоря, если у нас 10 пользователей и только один из них без зашифрованного пароля, то скрипт загрузит всех десятерых, зашифрует пароль у одного и сохранит всех десятерых обратно

Всё верно, будет шифровать только не зашифрованные пароли.
По окончании действия с результатом 0 сделать простое уведомление, что все не зашифрованные пароли хранимые в конфигурации зашифрованы. Если статус >0 то уведомление с ошибкой, что задача не выполнена.
И после этого действия заново считать конфигурацию.

> > > В конфигурационном файле нет открытых паролей для шифрования. > > > Не получится, команда > /usr/lib/ublinux/functions globalconf_convert_pass_plain_to_hash > возвращает 0 (успешное выполнение) даже если незашифрованных паролей в конфиге не было. Судя по выводу, этот скрипт загружает все параметры с паролями без проверки хэш это или пароль, проверяет его только перед тем как зашифровать пароль и перезаписывает всё что загружал в конфиг. Проще говоря, если у нас 10 пользователей и только один из них без зашифрованного пароля, то скрипт загрузит всех десятерых, зашифрует пароль у одного и сохранит всех десятерых обратно Всё верно, будет шифровать только не зашифрованные пароли. По окончании действия с результатом 0 сделать простое уведомление, что все не зашифрованные пароли хранимые в конфигурации зашифрованы. Если статус >0 то уведомление с ошибкой, что задача не выполнена. И после этого действия заново считать конфигурацию.
anshn commented 6 months ago
Poster
Collaborator

Для окна "Зашифровать все пароли" изменить предупреждение:

Внимание! Все незашифрованные пароли, хранящиеся в конфигурации, будут зашифрованы.
Вы уверены, что хотите зашифровать все пароли? Это действие нельзя отменить.

Для уведомления об успешном шифровании:

Все незашифрованные пароли были успешно зашифрованы.

Для уведомления о неудаче (или если есть привычная заглушка-текст, то использовать ее):

Ошибка! Задача не была выполнена.

Если нет возможности проверять результат выполнения, то общее уведомление:

Операция завершена.

Для окна "Зашифровать все пароли" изменить предупреждение: > **Внимание!** Все незашифрованные пароли, хранящиеся в конфигурации, будут зашифрованы. > Вы уверены, что хотите зашифровать все пароли? Это действие нельзя отменить. Для уведомления об успешном шифровании: > Все незашифрованные пароли были успешно зашифрованы. Для уведомления о неудаче (или если есть привычная заглушка-текст, то использовать ее): > Ошибка! Задача не была выполнена. Если нет возможности проверять результат выполнения, то общее уведомление: > Операция завершена.
Collaborator

Всё верно, будет шифровать только не зашифрованные пароли.
По окончании действия с результатом 0 сделать простое уведомление, что все не зашифрованные пароли хранимые в конфигурации зашифрованы. Если статус >0 то уведомление с ошибкой, что задача не выполнена.
И после этого действия заново считать конфигурацию.

Возвращается 0 даже если не было ничего зашифровано потому что незашифрованных не было. Тем не менее, он и без того зашифрованные пароли пересохраняет

> Всё верно, будет шифровать только не зашифрованные пароли. > По окончании действия с результатом 0 сделать простое уведомление, что все не зашифрованные пароли хранимые в конфигурации зашифрованы. Если статус >0 то уведомление с ошибкой, что задача не выполнена. > И после этого действия заново считать конфигурацию. Возвращается 0 даже если не было ничего зашифровано потому что незашифрованных не было. Тем не менее, он и без того зашифрованные пароли пересохраняет
Owner

Если нет НЕ ЗАШИФРОВАННЫХ - это не ошибка, с чего должен статус ошибки быть ?
Процесс выполнен -> статус 0 -> выводишь сообщение, всё логично, что не понятно ?

Если нет НЕ ЗАШИФРОВАННЫХ - это не ошибка, с чего должен статус ошибки быть ? Процесс выполнен -> статус 0 -> выводишь сообщение, всё логично, что не понятно ?
Collaborator

Жесть бесполезная кнопка.

Значит только статус какой то добавить бы, что шифровать нечего и что шифрование выполнено успешно. Например предлагаю:

Шифрование паролей выполнено успешно.

В конфигурационном файле нет открытых паролей для шифрования.

> Жесть бесполезная кнопка. > > Значит только статус какой то добавить бы, что шифровать нечего и что шифрование выполнено успешно. Например предлагаю: > > > Шифрование паролей выполнено успешно. > > > В конфигурационном файле нет открытых паролей для шифрования.
Owner

Для кнопки "Зашифровать все пароли":

  1. Добить тултип с расширенным описанием как в задаче дизайна
  2. По окончании действия с результатом 0 сделать простое уведомление, что все не зашифрованные пароли хранимые в конфигурации зашифрованы. Если статус >0 то уведомление с ошибкой, что задача не выполнена. Текст уведомления в ветке дизайна согласовать с Настей.
    После заново считать конфигурацию.
Для кнопки "Зашифровать все пароли": 1. Добить тултип с расширенным описанием как в задаче дизайна 2. По окончании действия с результатом 0 сделать простое уведомление, что все не зашифрованные пароли хранимые в конфигурации зашифрованы. Если статус >0 то уведомление с ошибкой, что задача не выполнена. Текст уведомления в ветке дизайна согласовать с Настей. После заново считать конфигурацию.
Collaborator
https://gitea.ublinux.ru/YanTheKaller/ubl-settings-usergroups/commit/1dae1c72eed79d89fe3563a6e6d48eaa7752a21f
anshn commented 6 months ago
Poster
Collaborator

Заголовок окна и сам текст предупреждения надо поправить

image

Заголовок окна и сам текст предупреждения надо поправить ![image](/attachments/057abfd3-104f-4539-81fb-bd39935c3345)
Collaborator
https://gitea.ublinux.ru/YanTheKaller/ubl-settings-usergroups/commit/adf3b045a4fbd5e54ee4210f16bdb46f5f4aa46a
anshn commented 6 months ago
Poster
Collaborator

Проверила, закрываю

Проверила, закрываю
anshn closed this issue 6 months ago
Sign in to join this conversation.
No Milestone
No project
3 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: Applications/ubl-settings-usergroups#243
Loading…
There is no content yet.