more use of None fixes

plone · Nov 23, 2016 · d3aa655 · d3aa655
1 parent aeadaef
commit d3aa655
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/src/plone.server/plone/server/auth/__init__.py b/src/plone.server/plone/server/auth/__init__.py
@@ -14,7 +14,7 @@ async def authenticate_request(request):
                         policy.name not in validator.for_validators):
                     continue
                 user = await validator(request).validate(token)
-                if user:
+                if user is not None:
                     return user
 
 

diff --git a/src/plone.server/plone/server/auth/participation.py b/src/plone.server/plone/server/auth/participation.py
@@ -29,7 +29,7 @@ async def __call__(self):
         # Cached user
         if not hasattr(self.request, '_cache_user'):
             user = await authenticate_request(self.request)
-            if user:
+            if user is not None:
                 self.request._cache_user = user
                 self.principal = user
         else:

diff --git a/src/plone.server/plone/server/auth/validators.py b/src/plone.server/plone/server/auth/validators.py
@@ -72,7 +72,7 @@ async def validate(self, token):
                 algorithms=[app_settings['jwt']['algorithm']])
             token['id'] = validated_jwt['id']
             user = await find_user(self.request, token)
-            if user and user.id == token['id']:
+            if user is not None and user.id == token['id']:
                 return user
         except jwt.exceptions.DecodeError:
             pass