(fix) extend verification token service to remove disabled check
Description
- If a user is not enabled, token check will always fail: https://github.com/nextcloud/server/blob/f61ef6d7e6a0480e88f8d7a18a514c9109103499/lib/private/Security/VerificationToken/VerificationToken.php#L46
- In the case of recovery email verification link for disabled users, it will therefore never actually work
Screenshots
Technical details
- I extend the verification token class and change the
checkmethod to remove the user enabled check
Tests
Test case 1: For a user disabled due to recovery email issue, verification link works and user is enabled ✅
Issues
10 commandments of code review
Edited by Akhil