Skip to content
This repository was archived by the owner on Jun 23, 2023. It is now read-only.

Commit a3b4efc

Browse files
committed
Verify that name is alphanum+space & remove condition in catch
1 parent 9836253 commit a3b4efc

File tree

2 files changed

+2
-15
lines changed

2 files changed

+2
-15
lines changed

logic/auth.js

Lines changed: 1 addition & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,6 @@ async function changeName(name) {
5454

5555
changeNameStatus.percent = 40; // eslint-disable-line no-magic-numbers
5656

57-
let complete = false;
5857
try {
5958
// get user data
6059
const user = await diskLogic.readUserFile();
@@ -65,25 +64,12 @@ async function changeName(name) {
6564
// update user file
6665
await diskLogic.writeUserFile({ ...user });
6766

68-
complete = true;
69-
7067
changeNameStatus.percent = 100;
7168
} catch (error) {
72-
if (error.response.status === constants.STATUS_CODES.UNAUTHORIZED) {
73-
changeNameStatus.unauthorized = true;
74-
} else {
75-
changeNameStatus.error = true;
76-
changeNameStatus.percent = 100;
77-
78-
throw error;
79-
}
80-
}
81-
82-
if (!complete) {
8369
changeNameStatus.error = true;
8470
changeNameStatus.percent = 100;
8571

86-
throw new Error('Unable to change name');
72+
throw error;
8773
}
8874
}
8975

routes/v1/account.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@ router.post('/change-name', auth.jwt, safeHandler(async (req, res, next) => {
1818

1919
try {
2020
validator.isString(newName);
21+
validator.isAlphanumericAndSpaces(newName);
2122
} catch (error) {
2223
return next(error);
2324
}

0 commit comments

Comments
 (0)