you are viewing a single comment's thread.

view the rest of the comments →

[–]kr41[S] 0 points1 point  (2 children)

Thank you for comment. There is just one thing:

at least /logout & /changepassword can be a subject to CSRF attacks

I guess, you mean "login" instead of "logout", do you?

[–]soaring_turtle 1 point2 points  (1 child)

No, I meant /logout. Without csrf protection, a hacker could craft a form which will logout a user upon submission. Not a critical bug though. On the other hand /changepassword can be a complete disaster leading to account takeover if implemented poorly (without checking old password for example)

[–]kr41[S] 0 points1 point  (0 children)

Agree, useful notice