Опыт: когда имперсонация автоматически завершается через некоторое время

[Moin]

Продолжение обсуждения темы Новая ошибка в экспериментальном интерфейсе имперсонации:

@Moin

Я заметил нечто похожее и думаю, что это было связано с тем, что я делал что-то ещё в промежутке, поэтому я имперсонировал пользователя дольше 15 минут. Мне кажется, что имперсонация в таком случае завершается, но интерфейс этого не отражает. После перезагрузки я всегда возвращался к своей учётной записи администратора.

@martin

Да, существует скрытая настройка experimental_impersonation_time_limit_minutes, установленная на 15 минут. Я не очень хорошо знаком с этой системой, но определённо было бы удобнее, если бы мы автоматически обновляли интерфейс после превышения этого лимита, или хотя бы указывали об этом где-нибудь.

@Moin

Не уверен, что это было связано с истечением 15 минут, но это не исключено. Я не хотел ждать ещё 15 минут для проверки. Меня больше интересовало, как редактирование категорий работает для модератора, чем сама функция имперсонации.

Но я думаю, что ошибка в консоли браузера была связана с таймером.

Изменение теперь стабильно, и я всё ещё могу воспроизвести эту проблему. Есть ли планы улучшить опыт работы администраторов после истечения срока имперсонации?

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

Также сбивает с толку ситуация, когда кнопка для прекращения имперсонации не работает.

Пример скриншота, где, похоже, пользователь может видеть категорию персонала (Команда) и видит ошибку при попытке использовать кнопку «Остановить имперсонацию».

[martin (Martin Brennan)]

Ах, моя вина, я забыл об этом, назначаю себя на исправление. Думаю, это две вещи:

1. Когда таймер истекает, отправить обновление MessageBus в UI, чтобы показать, что лимит времени истек. Возможно, даже показать таймер в UI…
2. Если нажать «Прекратить имитацию» и лимит времени истек, не выдавать ошибку, а просто перезагрузить страницу.

Может, увеличим время до 30 минут, 15 — это довольно быстро.

[ted (Ted Johansson)]

@martin

Может, увеличить время до 30 минут? 15 минут — это довольно быстро.

Как вы думаете, должно ли это быть 15 минут бездействия, а не обратный отсчёт, начинающийся при начале имитации?

Если вы ещё не приступили, я могу взглянуть на это.

[martin (Martin Brennan)]

Не уверен, просто интересно, как мы определим бездействие, если уже есть способ это узнать? Также не уверен в первоначальной идее таймера: был ли он предназначен для того, чтобы администраторы не оставались залогиненными слишком долго в чужие аккаунты? Не приведет ли возможность неограниченного откладывания таймера к нарушению этой цели? Не знаю…

Если мы это проясним, то, конечно, я с радостью передам это вам, если вы готовы сделать pull request

[ted (Ted Johansson)]

@martin

Также я не уверен в первоначальной идее таймера: была ли она в том, чтобы администраторы не оставались залогиненными слишком долго в чужие аккаунты? Не приведёт ли возможность бесконечной задержки таймера к нарушению этой цели? Не уверен…

Да. Это было до моего появления, поэтому сложно сделать выводы. Но поскольку длительность настраивается, возможно, лучше не трогать эту настройку.

@martin

Если мы сможем это выяснить, то, конечно, я с радостью передам это вам, если вы готовы сделать PR

Я могу начать с улучшения UX для существующего функционала, добавив более понятные сообщения.