search

dorinclisu / fastapi-auth0

FastAPI authentication and authorization using auth0.com
MIT License
230 stars 37 forks source link

Fix/missing kid #31

Closed theRealSuperMario closed 1 year ago

dorinclisu commented 1 year ago

Ok so I made a compromise with the error message to be "Malformed token header". The reason to include "header" is just for the sake of differentiation during tests, as I noticed during more experimentation that in some circumstances the reason for 401 could get mixed and we would get a false sense of security about the test case.

MaximilianFranz commented 1 year ago

Will there be a new release triggered containing this fix? If so, when? :) Thanks already for the work @theRealSuperMario and review @dorinclisu !

dorinclisu commented 1 year ago

Will release asap (next week at latest), the problem now is that mypy suddenly decided to fail my pipeline, no idea why at the moment since locally mypy doesn't return any errors for me.

dorinclisu commented 1 year ago

Released https://github.com/dorinclisu/fastapi-auth0/releases/tag/0.4.0