-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Labels
Description
Problem
Our login does not implement CSRF, since the user is not authenticated yet. This can be a problem, but the attack method is complex enough that bad actors will have to be highly educated, and realistically the site won't garner enough attention immediately to implement this. In the future it could require it though.
The two routes that should be protected with CSRF, but aren't are /register and /login. The way to potentially protect it would be pre-session tokens. This would also mitigate against session fixation attacks.
Reactions are currently unavailable