jazzband/djangorestframework-simplejwt

Implement django authentication signals

ricardocamposilva opened this issue · 2 comments

Wouldn't it be better?

If implemented this feature developers could apply their own logic around it and use where they want.
In this case I think it's more for security and audit purposes, today I have to apply lots of code lines to record a log raster.

This #196 should to it, although I wonder what should happen on token refresh (impressive job, by the way, @davesque)

As I see it, it should:

Emit logged_in if refresh OK
Emit login_failed if token is invalid
Emit logged_out if token is valid but expired

Roughly this: sieira#1 (the latest is a proposition, not a real pull request, since I doubt we can centralise the signal handling like that )

This would be a solid addition and I just came here looking to see if this was already supported. 👍