It would be good to have the possibility to disallow users to access Key authentication and generate keys themself.
The key has to be generated by the administrator.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | key_auth-permissions-3261708-3.patch | 2.66 KB | kiseleva.t |
Issue fork key_auth-3261708
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
Comments
Comment #3
kiseleva.t commentedComment #4
rajeshreeputra@kiseleva.t, This make sense.
I'm keeping this in need review(for next couple of days), so we can get few more insights, then will incorporate these changes in next release.
Comment #5
leksat commentedI found this issue for a bit different reason. On my website there are users with
administer userspermission. Yet I don't want them to manage the auth keys.Basically I need the following:
If this sounds like a good idea, we could add an update hook that adds the new
manage any auth keyspermission to all users havingadminister users.Comment #6
solideogloria commentedEither is fine for me, but I would like a permission like this added.
Comment #7
solideogloria commentedWe should target the latest branch.
(Also, could a maintainer add "2.1.x-dev" to the Version dropdown, please?)
Comment #8
rajeshreeputraCreated 2.1.x branch updated the same in MR, Thank You!!
Comment #9
reinimax commentedThanks for the patch, this is a very useful feature indeed!
However, I'm wondering whether it is a good idea to let any user with the "administer users" permission view the key authentication tab? Sure, they cannot edit the authentication key without the appropriate permission, but they can still see and potentially disclose or abuse this information.
In my opinion, it would be better to remove the "administer users" permission from the access check in UserKeyAuthForm, so that access is only granted if the user has the appropriate permissions from the key_auth module. If a role who manages users needs to view the key authentication tab, they can simply be assigned an additional role.
Comment #12
super_romeo commentedI added branch "3261708-a-separate-permission-" (please rename it to "3261708-a-separate-permission-without-administer-users") same to "3261708-a-separate-permission". MR18.
Comment #13
nickdjmWhat's holding this back from being merged in? If there's something more to be done I'd love to help push this through.
Comment #14
solideogloria commentedThe issue is marked Needs Review. Please test and review the changes in MR !18.
You can download the code changes as a patch/diff, and include the downloaded file in your Composer patches file.
Comment #16
rajeshreeputraMerged in 2.x branch!!