AuthenticationSystem::verifyUser now returns the user's Account instead of a boolean

This commit is contained in:
Benjamin Elsdon 2022-05-17 20:18:12 +08:00 committed by Melledy
parent fb93f1517c
commit b0aa294c26
2 changed files with 8 additions and 6 deletions

View File

@ -1,5 +1,6 @@
package emu.grasscutter.auth;
import emu.grasscutter.game.Account;
import emu.grasscutter.server.http.objects.*;
import express.http.Request;
import express.http.Response;
@ -30,10 +31,10 @@ public interface AuthenticationSystem {
/**
* Called by plugins to internally verify a user's identity.
* @param details A unique, one-time token to verify the user.
* @return True if the user is verified, False otherwise.
* @param details A unique identifier to identify the user. (For example: a JWT token)
* @return The user's account if the verification was successful, null if the user was unable to be verified.
*/
boolean verifyUser(String details);
Account verifyUser(String details);
/**
* This is the authenticator used for password authentication.

View File

@ -2,6 +2,7 @@ package emu.grasscutter.auth;
import emu.grasscutter.Grasscutter;
import emu.grasscutter.auth.DefaultAuthenticators.*;
import emu.grasscutter.game.Account;
import emu.grasscutter.server.http.objects.ComboTokenResJson;
import emu.grasscutter.server.http.objects.LoginResultJson;
@ -26,11 +27,11 @@ public final class DefaultAuthentication implements AuthenticationSystem {
public void resetPassword(String username) {
// Unhandled. The default authenticator doesn't store passwords.
}
@Override
public boolean verifyUser(String details) {
public Account verifyUser(String details) {
Grasscutter.getLogger().info(translate("dispatch.authentication.default_unable_to_verify"));
return false;
return null;
}
@Override