search

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 652 forks source link

Added JWT_PAYLOAD_HANDLER usage example to the docs #389

Open MauScheff opened 7 years ago

MauScheff commented 7 years ago

Added JWT_PAYLOAD_HANDLER usage example to the docs. https://github.com/GetBlimp/django-rest-framework-jwt/issues/367#issuecomment-334878640

@blueyed

codecov[bot] commented 7 years ago

Codecov Report

Merging #389 into master will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #389   +/-   ##
=======================================
  Coverage   90.67%   90.67%           
=======================================
  Files          14       14           
  Lines         847      847           
  Branches       29       29           
=======================================
  Hits          768      768           
  Misses         66       66           
  Partials       13       13
Flag Coverage Δ
#codecov 90.67% <ø> (ø) :arrow_up:
#dj110 87.48% <ø> (ø) :arrow_up:
#dj111 87.48% <ø> (ø) :arrow_up:
#dj18 89.84% <ø> (ø) :arrow_up:
#dj19 89.84% <ø> (ø) :arrow_up:
#drf31 89.84% <ø> (ø) :arrow_up:
#drf32 89.84% <ø> (ø) :arrow_up:
#drf33 89.84% <ø> (ø) :arrow_up:
#drf34 90.67% <ø> (ø) :arrow_up:
#drf35 90.31% <ø> (ø) :arrow_up:
#drf36 90.31% <ø> (ø) :arrow_up:
#py27 90.67% <ø> (ø) :arrow_up:
#py33 89.49% <ø> (ø) :arrow_up:
#py34 89.49% <ø> (ø) :arrow_up:

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...1300b5d. Read the comment docs.

blueyed commented 7 years ago

Reminded me of https://github.com/GetBlimp/django-rest-framework-jwt/pull/370.

blueyed commented 7 years ago

Maybe we should have an explit test to cover this use case?