javahuang / SurveyKing

Make a better survey system.
https://surveyking.cn
MIT License
3.26k stars 499 forks source link

Deactivate user not effect #58

Open menghaining opened 5 months ago

menghaining commented 5 months ago

Description

SurveyKing deactivate user not effect. After admin deactivating user, user can still operate using old session.

Affected version

v1.3.1, also the version in the latest docker version

updated at Nov 10, 2022

Attack vector(s)

Attacker obtains user1's sessionid. Admin deactivate user1. Attacker can continue attacking.

POC

  1. user1 login.

    image
  2. admin login and deactivate user1.

    image
  3. user1 can still operate by crafting request with old session.

    image