Только ваши заметки

Часть раздела Аутентификация и безопасность путешествия по Node.js на Coddy. Урок 18 из 24.

Вход в систему подтверждает, кто вы, но не то, что заметка принадлежит вам. GET /notes/:id ищет заметку только по её id, поэтому любой вошедший в систему, у которого есть id из общей ссылки или журнала, может прочитать эту заметку. Проверка владельца называется авторизацией.

Укажите владельца прямо в запросе, чтобы база данных никогда не возвращала чужой документ. Отвечайте 404, а не 403, если документ не найден: код 403 сообщит посетителю, что заметка существует.

const order = await orders.findOne({
    _id: new ObjectId(req.params.id),
    customer: req.user.id,
});
if (!order) {
    return res.status(404).json({ error: 'Order not found' });
}

Для каждого запроса, который обращается к данным пользователя, нужен фильтр: для чтения, обновления и удаления.

Задание

Легко

В этой главе мы возвращаемся к входу с помощью сессии из главы 2, а сервер стал больше. Теперь на нём есть страница по адресу GET /, маршруты для одной заметки GET /notes/:id и DELETE /notes/:id, а также учётная запись администратора. В двух маршрутах /notes/:id есть уязвимость: они находят заметку только по её идентификатору. Любой вошедший пользователь, знающий идентификатор, может прочитать или удалить чужую заметку. Устраните эту уязвимость.

В server.js, в маршруте GET /notes/:id, ищите заметку одновременно по её _id и owner. Владельцем должен быть вошедший пользователь — req.user.id.
Сделайте то же самое в DELETE /notes/:id, чтобы никто не мог удалить чужую заметку.
Если ни одна заметка не найдена, оба маршрута уже отвечают статусом 404 и {"error": "Note not found"}. Оставьте такой ответ и для чужой заметки. Ответ 403 сообщил бы посетителю, что заметка существует.

Попробуйте: нажмите «Играть».

Отправьте POST /signup, затем POST /login, указав в обоих запросах {"username": "ada", "password": "correct horse"}.
Отправьте POST /notes с {"text": "Safe code 4417"} и скопируйте _id из ответа. Запрос GET /notes/ с этим идентификатором в конце вернёт заметку.
Зарегистрируйтесь и войдите в систему с {"username": "bob", "password": "correct horse"}. Отправьте GET /notes/ с идентификатором Ады в конце: получите статус 404 и {"error": "Note not found"}. Отправьте DELETE /notes/ с тем же идентификатором в конце: также получите 404.
Снова войдите в систему как Ада, указав данные из первого шага. Её заметка всё ещё на месте. Теперь удалите её с помощью DELETE: статус 204.

Попробуйте сами

import express from 'express';
import { ObjectId } from 'mongodb';
import { randomBytes } from 'node:crypto';
import { parseCookies } from './cookies.js';
import { notes, sessions, users } from './db.js';
import { hashPassword, verifyPassword } from './passwords.js';

const app = express();
app.use(express.json());

// Единственная учётная запись администратора. Её пароль берётся из переменных окружения.
await users.insertOne({ username: 'admin', role: 'admin', passwordHash: await hashPassword(process.env.ADMIN_PASSWORD) });

app.post('/signup', async (req, res) => {
    const username = String(req.body.username ?? '').trim().toLowerCase();
    const password = String(req.body.password ?? '');
    if (!username || password.length < 8) {
        return res.status(400).json({ error: 'Pick a username and a password of at least 8 characters' });
    }
    if (await users.findOne({ username })) {
        return res.status(409).json({ error: 'That username is taken' });
    }
    await users.insertOne({ username, passwordHash: await hashPassword(password) });
    res.status(201).json({ username });
});

app.post('/login', async (req, res) => {
    const username = String(req.body.username ?? '').trim().toLowerCase();
    const user = await users.findOne({ username });
    if (!user || !(await verifyPassword(String(req.body.password ?? ''), user.passwordHash))) {
        return res.status(401).json({ error: 'Wrong username or password' });
    }
    const sid = randomBytes(24).toString('base64url');
    await sessions.insertOne({ _id: sid, userId: user._id, createdAt: new Date() });
    res.cookie('sid', sid, {
        httpOnly: true,
        sameSite: 'lax',
        secure: true,
        maxAge: 24 * 60 * 60 * 1000,
    });
    res.json({ username: user.username });
});

// Находит пользователя по сеансовому cookie или возвращает 401.
async function requireUser(req, res, next) {
    const { sid } = parseCookies(req.get('cookie'));
    const session = sid && (await sessions.findOne({ _id: sid }));
    const user = session && (await users.findOne({ _id: session.userId }));
    if (!user) {
        return res.status(401).json({ error: 'Log in first' });
    }
    req.user = { id: user._id.toHexString(), username: user.username, role: user.role };
    next();
}

app.get('/me', requireUser, (req, res) => {
    res.json({ username: req.user.username });
});

app.post('/logout', async (req, res) => {
    const { sid } = parseCookies(req.get('cookie'));
    if (sid) {
        await sessions.deleteOne({ _id: sid });
    }
    res.clearCookie('sid');
    res.status(204).end();
});

// Ваши заметки в виде страницы.
app.get('/', requireUser, async (req, res) => {
    const list = await notes.find({ owner: req.user.id }).toArray();
    const items = list.map((note) => `<li>${note.text}</li>`).join('');
    res.send(`<!doctype html><title>Vault</title><h1>${req.user.username}'s notes</h1><ul>${items}</ul>`);
});

// Заметки: каждый пользователь читает и записывает только свои.
app.get('/notes', requireUser, async (req, res) => {
    res.json(await notes.find({ owner: req.user.id }).toArray());
});

app.post('/notes', requireUser, async (req, res) => {
    const note = { owner: req.user.id, text: String(req.body.text ?? '') };
    await notes.insertOne(note);
    res.status(201).json(note);
});

app.get('/notes/:id', requireUser, async (req, res) => {
    if (!ObjectId.isValid(req.params.id)) {
        return res.status(404).json({ error: 'Note not found' });
    }
    const note = await notes.findOne({ _id: new ObjectId(req.params.id) });
    if (!note) {
        return res.status(404).json({ error: 'Note not found' });
    }
    res.json(note);
});

app.delete('/notes/:id', requireUser, async (req, res) => {
    if (!ObjectId.isValid(req.params.id)) {
        return res.status(404).json({ error: 'Note not found' });
    }
    const { deletedCount } = await notes.deleteOne({ _id: new ObjectId(req.params.id) });
    if (!deletedCount) {
        return res.status(404).json({ error: 'Note not found' });
    }
    res.status(204).end();
});

// Для администраторов: сколько всего учётных записей.
app.get('/admin', requireUser, async (req, res) => {
    res.json({ users: await users.countDocuments() });
});

app.listen(3000, () => {
    console.log('Vault is running on port 3000');
});