Odświeżanie odpalonego JSON serwera

Question

Cześć,

Mam problem w mojej aplikacji napisanej w React. Zrobiłem panel rejestracji, rejestruję użytkownika to widzę, że poprawnie dodaje go do listy użytkowników w users.json. Problem jest w tym, że nie mogę od razu się na to świeżo stworzone konto użytkownika zalogować. Mogę się zalogować jedynie na użytkowników, którzy byli w pliku users.json przed odpaleniem serwera, w trakcie gdy serwer jest odpalony nowych użytkowników nie mogę się zalogować.

Może jest jakiś sposób, na odświeżanie serwera po każdej rejestracji?

Za wszelkie rady dziękuję! ;)

Comments

A jak masz zaimplementowaną rejestracje i logowanie?

---

Rejestracje i logowanie mam stworzone jako komponenty oddzielnie w aplikacji react, wartości z rejestracji przesyłam na serwer i tam w pliku server.js pushuje te dane do pliku users.json

---

Czy możesz pokazać implementację backendu? Bo po samym opisie trudno jest wydedukować, gdzie może leżeć problem. Zgadywałbym, że może nie synchronizujesz danych z pliku JSON z nowo zarejestrowanymi użytkownikami, więc nie są oni widoczni przy logowaniu.

---

server.post('/api/auth/register', (req, res) => {
	const { email, password, username } = req.body;
	if (isRegisterAuthenticated({ email })) {
		const status = 401;
		const message = 'Email already exist';
		res.status(status).json({ status, message });
		return;
	}

	fs.readFile('./users.json', (err, data) => {
		if (err) {
			const status = 401;
			const message = err;
			res.status(status).json({ status, message });
			return;
		}
		data = JSON.parse(data.toString());
		const last_item_id = data.users[data.users.length - 1].id;

		data.users.push({
			id: last_item_id + 1,
			email: email,
			password: password,
			username: username,
			favorites: [],
		});
		fs.writeFile('./users.json', JSON.stringify(data), (err, result) => {
			if (err) {
				const status = 401;
				const message = err;
				res.status(status).json({ status, message });
				return;
			}
		});
	});
	// const access_token = createToken({email, password, username});
	res.status(200).json({ status: 'Success' });
});

server.post('/api/auth/login', (req, res) => {
	const { email, password } = req.body;
	const user = isLoginAuthenticated({ email, password }).pop();
	console.log(user);

	if (!user) {
		const status = 401;
		const message = 'Incorrect Email or Password';
		res.status(status).json({ status, message });
		return;
	}
	const access_token = createToken({ email, password });
	res.status(200).json({ access_token, userId: user.id });
});

To jest kod z server.js

---

Na oko, masz race condition wewnątrz middleware rejestracji: fs.readFile oraz fs.writeFile są asynchroniczne, więc zanim wykonają się przekazane do nich callbacki, to cały middleware odpowiedzialny za rejestrację zdąży odesłać odpowiedź wykonując ostatnią linijkę z res.status( .. ).json( .. ). Nie wiem na ile to wpływa na cały proces - bo te operacje na plikach nie powinny być na tyle wolne, żeby (bez automatyzacji) można było zdążyć wyklikać logowanie nowo zarejestrowanego użytkownika. Jest natomiast ryzyko, że rejestracja odpowie "Success", zanim ten proces faktycznie się skończy. Użyłbym tutaj synchronicznych odpowiedników metod: fs.readFileSync i fs.writeFileSync, albo - jeśli potrzebujesz asynchroniczności - odpowiedników opartych o promise; łatwiej będzie wtedy kontrolować przepływ kodu.

Pokaż też implementację funkcji isLoginAuthenticated - jak ona bierze usera?

---

function isLoginAuthenticated({ email, password }) {
	return userdb.users.filter(
		(user) => user.email === email && user.password === password
	);
}

 

---

Skąd jest brana tablica userdb.users?

---

const server = jsonServer.create();
const userdb = JSON.parse(fs.readFileSync('./users.json', 'utf-8'));

---

Czyli plik users.json odczytujesz tylko raz, podczas startu serwera i nigdzie nie aktualizujesz zmiennej userdb? Jeśli tak, to - jak dedukowałem wcześniej - operujesz na początkowych danych użytkowników. Więc przy logowaniu nie ma dostępu do nowych.

---

To za każdym razem powielać to linijkę kodu kiedy wyciągam dane z users.json? Czy jest jakiś lepszy sposób?

const userdb = JSON.parse(fs.readFileSync('./users.json', 'utf-8'));

 

Answer 1

Jeśli proces logowania ma mieć dostęp do aktualnych użytkowników, to albo za każdym razem odczytuj plik users.json (tak jak robisz to przy rejestracji; tylko popraw możliwy race condition) albo (tak jak teraz) wczytaj go raz (na starcie serwera), sparsuj i na tym operuj w middleware - żeby zawartość była ciągle aktualna. Zapis do pliku możesz zrobić wtedy, gdy serwer będzie wyłączany (możesz się podpiąć pod exit event - pamiętając, że tam można wykonywać tylko operacje synchroniczne).

Natomiast, w dłuższej perspektywie lepiej skorzystać z bazy danych.

Comments

Bardzo, dziękuję Ci za twoją za Pomoc! :)