[UX] Improve untrusted user warning message#15369
[UX] Improve untrusted user warning message#15369lisanna-dettwyler wants to merge 1 commit intoNixOS:masterfrom
Conversation
cefd115 to
90470c7
Compare
|
| constexpr auto UNTRUSTED_MSG = "Add your username to the " ANSI_BOLD "trusted-users" ANSI_NORMAL | ||
| " setting in /etc/nix/nix.conf and then restart the daemon\n" |
There was a problem hiding this comment.
It might be better to have this link to a docs page about trusted users. There are caveats here: like the trust model, reasons to not do this, etc. that deserve to be presented. If the tool presents the simple instruction to follow, it could achieve it better by dropping untrusted users altogether (which would be a mistake.)
There was a problem hiding this comment.
Updated to do this, and added a note to trusted-users about alternatives.
|
Is it OK to recommend that someone trying to use a privileged setting to make themselves a trusted user? Or should the warning message more specifically recommend that they add the privileged setting itself to |
|
This should really be up to the user, especially because being in |
|
If anything, even less users should be specifying |
90470c7 to
43cc989
Compare
43cc989 to
6a51d4d
Compare
For some reason the warning message gets printed twice... not sure what's going on there. |
6a51d4d to
80168d9
Compare
Direct users to the correct section in the nix.conf man page to read about trusted users, as well as any privileged setting they are trying to set. Also improve the documentation for trusted-users regarding where it can be set, when it will be ignored, and alternatives to updating it. Resolves NixOS#8248 Signed-off-by: Lisanna Dettwyler <lisanna.dettwyler@gmail.com>
80168d9 to
0a3bddd
Compare
Based on user feedback mentioned in the linked issue, more information about how to resolve an untrusted user issue should be provided in the warning message.
Resolves #8248
Motivation
This resolves a popular UX issue.
Context
#8248
Add 👍 to pull requests you find important.
The Nix maintainer team uses a GitHub project board to schedule and track reviews.