Merge pull request #3144 from Meus-Artis/patch-1 Better console output

7f9766c9186296fe72da0eee12e6dc9bdd12d4ff

Cohee <18619528+Cohee1207@users.noreply.github.com>

Signed

src/endpoints/users-public.js+8 -8 Viewed

@@ -67,17 +67,17 @@ router.post('/login', jsonParser, async (request, response) => {
67	67	const user = await storage.getItem(toKey(request.body.handle));
68	68
69	69	if (!user) {
70	70	console.log('Login failed: User', request.body.handle, 'not found');
71	71	return response.status(403).json({ error: 'Incorrect credentials' });
72	72	}
73	73
74	74	if (!user.enabled) {
75	75	console.log('Login failed: User', user.handle, 'is disabled');
76	76	return response.status(403).json({ error: 'User is disabled' });
77	77	}
78	78
79	79	if (user.password && user.password !== getPasswordHash(request.body.password, user.salt)) {
80	80	console.log('Login failed: Incorrect password for', user.handle);
81	81	return response.status(403).json({ error: 'Incorrect credentials' });
82	82	}
83	83

         await loginLimiter.delete(ip);
         request.session.handle = user.handle;
         console.log('Login successful:', user.handle, request'from', ip, 'at', new Date().sessiontoLocaleString());
         return response.json({ handle: user.handle });
     } catch (error) {
         if (error instanceof RateLimiterRes) {

@@ -115,12 +115,12 @@ router.post('/recover-step1', jsonParser, async (request, response) => {
115	115	const user = await storage.getItem(toKey(request.body.handle));
116	116
117	117	if (!user) {
118	118	console.log('Recover step 1 failed: User', request.body.handle, 'not found');
119	119	return response.status(404).json({ error: 'User not found' });
120	120	}
121	121
122	122	if (!user.enabled) {
123	123	console.log('Recover step 1 failed: User', user.handle, 'is disabled');
124	124	return response.status(403).json({ error: 'User is disabled' });
125	125	}
126	126

@@ -153,12 +153,12 @@ router.post('/recover-step2', jsonParser, async (request, response) => {
153	153	const ip = getIpFromRequest(request);
154	154
155	155	if (!user) {
156	156	console.log('Recover step 2 failed: User', request.body.handle, 'not found');
157	157	return response.status(404).json({ error: 'User not found' });
158	158	}
159	159
160	160	if (!user.enabled) {
161	161	console.log('Recover step 2 failed: User', user.handle, 'is disabled');
162	162	return response.status(403).json({ error: 'User is disabled' });
163	163	}
164	164