entrouvert
/
debian-python-raven
17
0
Fork 0
Code Issues Pull Requests Releases Activity

Remove is_authenticated property (excl. Tornado)

This commit is contained in:
David Cramer 2015-09-10 16:07:29 -05:00
parent 24521704c6
commit 6a84d48564
3 changed files with 6 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
raven/contrib/django/client.py
View File

@ -35,11 +35,10 @@ class DjangoClient(Client):
def get_user_info(self, user):
if not user.is_authenticated():
return {'is_authenticated': False}
return {}
user_info = {
'id': user.pk,
'is_authenticated': True,
}
if hasattr(user, 'email'):

9
raven/contrib/zope/__init__.py
View File

@ -96,11 +96,12 @@ class ZopeSentryHandler(SentryHandler):
setattr(record, 'request', http)
user = request.get('AUTHENTICATED_USER', None)
if user is not None and user != nobody:
user_dict = dict(id=user.getId(),
is_authenticated=True,
email=user.getProperty('email') or '')
user_dict = {
'id': user.getId(),
'email': user.getProperty('email') or '',
}
else:
user_dict = {'is_authenticated': False}
user_dict = {}
setattr(record, 'user', user_dict)
except (AttributeError, KeyError):
logger.warning('Could not extract data from request', exc_info=True)

2
tests/contrib/django/tests.py
View File

@ -193,7 +193,6 @@ class DjangoClientTest(TestCase):
assert 'user' in event
user_info = event['user']
assert user_info == {
'is_authenticated': True,
'username': user.username,
'id': user.id,
'email': user.email,
@ -217,7 +216,6 @@ class DjangoClientTest(TestCase):
)
user_info = self.raven.get_user_info(user)
assert user_info == {
'is_authenticated': True,
'username': user.username,
'id': user.id,
'email': user.email,