plone / guillotina

Python AsyncIO data API to manage billions of resources

https://guillotina.readthedocs.io/en/latest/

Other

187 stars 51 forks source link

Also allow JWT sub claim #987

Closed allusa closed 4 years ago

allusa commented 4 years ago

Allow loginid to be defined in the sub (subject) JWT claim. Also maintain the private Guillotina id claim.

bloodbare commented 4 years ago

@allusa changelog needs to be changed and can you also add the sub at token generation ?

codecov-commenter commented 4 years ago

Codecov Report

Merging #987 into master will not change coverage. The diff coverage is 100.0%.

@@          Coverage Diff           @@
##           master    #987   +/-   ##
======================================
  Coverage    94.6%   94.6%           
======================================
  Files         316     316           
  Lines       28047   28047           
======================================
  Hits        26518   26518           
  Misses       1529    1529

Impacted Files	Coverage Δ
guillotina/auth/utils.py	`100.0% <ø> (ø)`
guillotina/auth/validators.py	`88.6% <100.0%> (ø)`