Fix code review suggestions

This commit is contained in:
Zaprit 2024-10-20 17:32:04 +01:00
commit d34e6a7b1c
2 changed files with 10 additions and 4 deletions

View file

@ -17,8 +17,14 @@ namespace LBPUnion.ProjectLighthouse.Servers.GameServer.Controllers.Login;
[ApiController]
[Route("LITTLEBIGPLANETPS3_XML/login")]
[Produces("text/xml")]
public class LoginController(DatabaseContext database) : ControllerBase
public class LoginController : ControllerBase
{
private readonly DatabaseContext database;
public LoginController(DatabaseContext database)
{
this.database = database;
}
[HttpPost]
public async Task<IActionResult> Login()
{
@ -51,13 +57,13 @@ public class LoginController(DatabaseContext database) : ControllerBase
string username = npTicket.Username;
if (String.IsNullOrEmpty(username))
if (string.IsNullOrEmpty(username))
{
Logger.Warn("Unable to determine username, rejecting login", LogArea.Login);
return this.Forbid();
}
await database.RemoveExpiredTokens();
await this.database.RemoveExpiredTokens();
UserEntity? user;

View file

@ -11,7 +11,7 @@ public class ServerConfiguration : ConfigurationBase<ServerConfiguration>
// This is so Lighthouse can properly identify outdated configurations and update them with newer settings accordingly.
// If you are modifying anything here, this value MUST be incremented.
// Thanks for listening~
public override int ConfigVersion { get; set; } = 26;
public override int ConfigVersion { get; set; } = 27;
public override string ConfigName { get; set; } = "lighthouse.yml";
public string WebsiteListenUrl { get; set; } = "http://localhost:10060";