Skip to content
This repository has been archived by the owner on Feb 22, 2024. It is now read-only.

Fails on authenticating dict based user #92

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

menudoproblema
Copy link

To reproduce this bug you can use this authenticate function:

def authenticate(username, password):
    return {'id': '1', 'username': 'test', 'password': 'test'}

Trace:

File ".../lib/python3.5/site-packages/flask_jwt/__init__.py", line 53, in _default_jwt_payload_handler
    identity = getattr(identity, 'id') or identity['id']
AttributeError: 'dict' object has no attribute 'id'

@coveralls
Copy link

coveralls commented Jul 10, 2016

Coverage Status

Coverage remained the same at 95.652% when pulling 030e870 on menudoproblema:master into c270841 on mattupstate:master.

@stephane
Copy link

stephane commented Aug 3, 2016

Already fixed by #63 and so many others ;)

Updated after @dequis comment

@dequis
Copy link

dequis commented Dec 13, 2016

@stephane I think you mean #63

@stephane
Copy link

Yes you're right

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants