Skip to content

Commit

Permalink
clarify impact of admin_users withotu specifying implementation detail
Browse files Browse the repository at this point in the history
  • Loading branch information
minrk committed Feb 12, 2024
1 parent 4874818 commit eb30a9b
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion docs/source/topic/allowing.md
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,7 @@ c.OAuthenticator.allowed_users = {"mensah", "ratthi"}

If this is your only configuration, only these users will be allowed, no others.

Note that any additional usernames in the deprecated `admin_users` configuration will also be added to the `allowed_users` set.
Note that any additional usernames in the deprecated `admin_users` configuration will also be allowed to login.

```{seealso}
Configuration documentation for {attr}`.OAuthenticator.allowed_users`
Expand Down
2 changes: 1 addition & 1 deletion oauthenticator/oauth2.py
Original file line number Diff line number Diff line change
Expand Up @@ -1093,5 +1093,5 @@ def __init__(self, **kwargs):
If unspecified, grants no access. You must set at least one other `allow` configuration
if any users are to have permission to access the Hub.
Any users in `admin_users` will be added to this set.
Any usernames in `admin_users` will also be allowed to login.
"""

0 comments on commit eb30a9b

Please sign in to comment.