Skip to content

Commit

Permalink
Minor tweaks
Browse files Browse the repository at this point in the history
  • Loading branch information
caspervdw committed Nov 14, 2023
1 parent 37a9d4e commit ceff950
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 5 deletions.
2 changes: 1 addition & 1 deletion README.rst
Original file line number Diff line number Diff line change
Expand Up @@ -131,7 +131,7 @@ First-time logins

For first-time logins, there is no RemoteUser object to match the external
user ID with a local django user. In this case, users are accepted only if the
user presents a valid invitation (or if there are AUTO_PERMISSIONS) setup, see below.
user presents a valid invitation (or when using ``AutoPermissionBackend``, see below).
This is because there is no way to safely
match external user ids to local django users.

Expand Down
5 changes: 1 addition & 4 deletions nens_auth_client/backends.py
Original file line number Diff line number Diff line change
Expand Up @@ -210,10 +210,7 @@ def authenticate(self, request, claims):
user or None
"""
provider_name = _extract_provider_name(claims)
if (
not provider_name
or provider_name not in settings.NENS_AUTH_AUTO_PERMISSIONS
):
if provider_name not in settings.NENS_AUTH_AUTO_PERMISSIONS:
return

# Create user and remote user
Expand Down

0 comments on commit ceff950

Please sign in to comment.