diff --git a/src/Identity/IdentityServer/RequestValidators/ClientVersionValidator.cs b/src/Identity/IdentityServer/RequestValidators/ClientVersionValidator.cs index 558ad041c2..378b272a62 100644 --- a/src/Identity/IdentityServer/RequestValidators/ClientVersionValidator.cs +++ b/src/Identity/IdentityServer/RequestValidators/ClientVersionValidator.cs @@ -30,9 +30,12 @@ public class ClientVersionValidator( public async Task ValidateAsync(User? user, CustomValidatorRequestContext requestContext) { + // Do this nullish check because the base request validator currently is not + // strict null checking. Once that gets fixed then we can see about making + // the user not nullish checked. If they are null then the validator should fail. if (user == null) { - return true; + return false; } Version? clientVersion = currentContext.ClientVersion;