accountStatus cannot both be inactive and disabled, changed to or condition. Also removed outdated comment.

This commit is contained in:
Nathan Byrd 2023-01-02 15:20:24 -06:00
parent 25e3630458
commit 395676f19d
1 changed files with 1 additions and 5 deletions

View File

@ -97,7 +97,6 @@ exports.getModule = class WebFingerServerModule extends WebHandlerModule {
); );
} }
// TODO: Handle URL escaped @ sign as well
const userPosition = resource.indexOf('@'); const userPosition = resource.indexOf('@');
if (-1 == userPosition || userPosition == resource.length - 1) { if (-1 == userPosition || userPosition == resource.length - 1) {
this._notFound(resp); this._notFound(resp);
@ -314,10 +313,7 @@ Achievement Points: %ACHIEVEMENT_POINTS%`,
} }
const accountStatus = user.getPropertyAsNumber(UserProps.AccountStatus); const accountStatus = user.getPropertyAsNumber(UserProps.AccountStatus);
if ( if (User.AccountStatus.disabled == accountStatus || User.AccountStatus.inactive == accountStatus) {
User.AccountStatus.disabled == accountStatus &&
User.AccountStatus.inactive == accountStatus
) {
this._notFound(resp); this._notFound(resp);
return cb( return cb(
Errors.AccessDenied('Account disabled', ErrorReasons.Disabled) Errors.AccessDenied('Account disabled', ErrorReasons.Disabled)