vinzenz/servicepoint-tanks
0
0
Fork 0
Code Issues Pull requests Actions Packages Projects Releases Wiki Activity

fix crypto.randomUUID() is not a function in an insecure context

Browse source
This commit is contained in:
Vinzenz Schroeter 2024-04-19 13:29:51 +02:00
parent 25a3adea2a
commit 61f90b99af
2 changed files with 6 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
TanksServer/Endpoints.cs
View file

@ -24,7 +24,10 @@ internal static class Endpoints
if (name.Length > 12) if (name.Length > 12)
return Results.BadRequest("name too long"); return Results.BadRequest("name too long");
var player = playerService.GetOrAdd(name, id ?? Guid.NewGuid()); if (!id.HasValue || id.Value == Guid.Empty)
id = Guid.NewGuid();
var player = playerService.GetOrAdd(name, id.Value);
return player != null return player != null
? Results.Ok(new NameId(player.Name, player.Id)) ? Results.Ok(new NameId(player.Name, player.Id))
: Results.Unauthorized(); : Results.Unauthorized();

3
tank-frontend/src/App.tsx
View file

@ -11,9 +11,10 @@ import Scoreboard from "./Scoreboard.tsx";
import Button from "./components/Button.tsx"; import Button from "./components/Button.tsx";
import './App.css'; import './App.css';
import {getRandomTheme, useStoredTheme} from "./theme.ts"; import {getRandomTheme, useStoredTheme} from "./theme.ts";
import {EmptyGuid} from "./Guid.ts";
const getNewNameId = () => ({ const getNewNameId = () => ({
id: crypto.randomUUID(), id: EmptyGuid,
name: '' name: ''
}); });