django-rest-framework-jwt
django-rest-framework-jwt copied to clipboard
Pass Request from JSONWebTokenSerializer to django.contrib.auth
Closes #387
Codecov Report
Merging #412 into master will decrease coverage by
0.26%. The diff coverage is92.85%.
@@ 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 dataPowered by Codecov. Last update 0a0bd40...a950756. Read the comment docs.
Any updates on this?
+1
+1
+1
@jpadilla is it possible to merge this PR into master?
@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.