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

update _default_jwt_payload_handler #103

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

update _default_jwt_payload_handler #103

wants to merge 1 commit into from

Conversation

buddiex
Copy link

@buddiex buddiex commented Oct 15, 2016

made changes to cater for when the instance of the return identity is a dictionary.
the current code returns the following error

  identity = getattr(identity, 'id') or identity['id']
AttributeError: 'dict' object has no attribute 'id'

@buddiex
update _default_jwt_payload_handler
801fcce 
made changes to cater for when the instance of the return identity is a dictionary. 
the current code returns the following error
```    identity = getattr(identity, 'id') or identity['id']
AttributeError: 'dict' object has no attribute 'id'
```
@coveralls
Copy link

coveralls commented Oct 15, 2016 •
edited
Loading

Coverage Status

Coverage remained the same at 95.652% when pulling 801fcce on buddiex:master into c270841 on mattupstate:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 95.652% when pulling 801fcce on buddiex:master into c270841 on mattupstate:master.

@buddiex
Copy link
Author

buddiex commented Oct 15, 2016

will close this...
I seams to already been taken care of by other pull request.
#63

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@buddiex @coveralls