search

deejungx / flask-cognito-extended

Flask implementation for Amazon Cognito
MIT License
6 stars 6 forks source link

Fix Wrong Token Expired Error Exception #7

Closed chia-labradorsystems closed 3 years ago

chia-labradorsystems commented 3 years ago

change JWTDecodeError to ExpiredSignatureError in order to trigger token expired callback correctly

codecov[bot] commented 3 years ago

Codecov Report

Merging #7 (95b2b14) into master (ddade34) will increase coverage by 0.22%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master       #7      +/-   ##
==========================================
+ Coverage   55.17%   55.40%   +0.22%     
==========================================
  Files           7        8       +1     
  Lines         783      787       +4     
  Branches       91       91              
==========================================
+ Hits          432      436       +4     
  Misses        349      349              
  Partials        2        2
Impacted Files Coverage Δ
flask_cognito_extended/tokens.py 16.32% <100.00%> (ø)
flask_cognito_extended/__init__.py 100.00% <0.00%> (ø)

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 ddade34...95b2b14. Read the comment docs.

deejungx commented 3 years ago

Works great. Thanks for noticing and fixing the error.