jpadilla / django-rest-framework-jwt

JSON Web Token Authentication support for Django REST Framework
http://jpadilla.github.io/django-rest-framework-jwt/
MIT License
3.19k stars 650 forks source link

Pass Request from JSONWebTokenSerializer to django.contrib.auth #412

Open jksimoniii opened 6 years ago

jksimoniii commented 6 years ago

Closes #387

codecov[bot] commented 6 years ago

Codecov Report

Merging #412 into master will decrease coverage by 0.26%. The diff coverage is 92.85%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #412      +/-   ##
==========================================
- Coverage   90.67%   90.41%   -0.27%     
==========================================
  Files          14       12       -2     
  Lines         847      824      -23     
  Branches       29       29              
==========================================
- Hits          768      745      -23     
  Misses         66       66              
  Partials       13       13
Flag Coverage Δ
#codecov 90.41% <92.85%> (-0.27%) :arrow_down:
#dj110 87.13% <85.71%> (-0.35%) :arrow_down:
#dj111 87.13% <85.71%> (-0.35%) :arrow_down:
#dj18 89.56% <85.71%> (-0.29%) :arrow_down:
#dj19 89.56% <85.71%> (-0.29%) :arrow_down:
#drf31 89.56% <85.71%> (-0.29%) :arrow_down:
#drf32 89.56% <85.71%> (-0.29%) :arrow_down:
#drf33 89.56% <85.71%> (-0.29%) :arrow_down:
#drf34 90.41% <92.85%> (-0.27%) :arrow_down:
#drf35 90.04% <92.85%> (-0.28%) :arrow_down:
#drf36 90.04% <92.85%> (-0.28%) :arrow_down:
#py27 90.41% <92.85%> (-0.27%) :arrow_down:
#py33 89.19% <85.71%> (-0.3%) :arrow_down:
#py34 90.04% <92.85%> (+0.55%) :arrow_up:
#py35 87.13% <85.71%> (?)
#py36 87.13% <85.71%> (?)
Impacted Files Coverage Δ
rest_framework_jwt/serializers.py 82.41% <100%> (ø) :arrow_up:
tests/test_serializers.py 95.71% <92.3%> (+0.4%) :arrow_up:
rest_framework_jwt/models.py
rest_framework_jwt/utils.py

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 0a0bd40...a950756. Read the comment docs.

Moulde commented 6 years ago

Any updates on this?

rubgombar1 commented 6 years ago

+1

mscansian commented 6 years ago

+1

JamesonNetworks commented 6 years ago

+1

mscansian commented 6 years ago

@jpadilla is it possible to merge this PR into master?

paymog commented 4 years ago

@jpadilla ping again, it's been almost 2 years since this was proposed. Is there anything I can do to speed things up?

EDIT: I just found https://github.com/jpadilla/django-rest-framework-jwt/issues/484. Guess this will never be merged.