diff options
| author | Madeline <46743919+MaddyUnderStars@users.noreply.github.com> | 2022-02-17 20:32:47 +1100 |
|---|---|---|
| committer | Madeline <46743919+MaddyUnderStars@users.noreply.github.com> | 2022-02-17 20:32:47 +1100 |
| commit | c23ec4aceebc5221aa5a366f408c4b709ca17578 (patch) | |
| tree | 0f9dc8aeb125488790591a7b5e835591280f2b94 | |
| parent | Merge pull request #608 from Featyre/master (diff) | |
| download | server-c23ec4aceebc5221aa5a366f408c4b709ca17578.tar.xz | |
Fixed bug in /users/@me PATCH where username must be present in every request, breaking account claiming
| -rw-r--r-- | api/src/routes/users/@me/index.ts | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/api/src/routes/users/@me/index.ts b/api/src/routes/users/@me/index.ts index acca8910..75c91001 100644 --- a/api/src/routes/users/@me/index.ts +++ b/api/src/routes/users/@me/index.ts @@ -58,7 +58,8 @@ router.patch("/", route({ body: "UserModifySchema" }), async (req: Request, res: } var check_username = body?.username?.replace(/\s/g, ''); - if(!check_username) { + //claiming an account does not provide username so check if username in body before throw + if (!check_username && body.username) { throw FieldErrors({ username: { code: "BASE_TYPE_REQUIRED", message: req.t("common:field.BASE_TYPE_REQUIRED") } }); |