입력값 검증

Coddy Node.js 여정의 REST API 섹션에 포함된 레슨. 25개 중 6번째.

클라이언트는 본문에 무엇이든 보낼 수 있으며, if (!name)은 빈 값만 걸러냅니다. 123, true, ["x"]는 참으로 평가되어 통과합니다. 공백으로만 이루어진 문자열도 통과합니다. ' '.trim()의 결과가 ''이므로 trim()으로 이를 걸러내세요. typeof로 각 필드의 유형을 확인한 다음, 범위를 검사하세요:

const { seats } = req.body ?? {};
if (typeof seats !== 'number' || !Number.isInteger(seats) || seats < 1) {
    return res.status(400).json({ error: 'seats must be a whole number from 1' });
}

응답을 읽는 사람을 위한 메시지를 작성하세요. 필드 이름을 말하고 어떤 값이어야 하는지 알려 주어 무엇을 수정해야 하는지 알 수 있게 하세요. PATCH 요청에서는 전송된 필드만 확인합니다. 나머지 필드는 변경되지 않기 때문입니다.

챌린지

쉬움

현재 if (!title)는 123이나 " " 같은 제목도 통과시킵니다. 저장하기 전에 각 필드의 유형과 크기를 확인하세요.

routes/tasks.js의 POST 라우트에서 title을 확인하세요:
- 문자열이어야 합니다.
- 빈 문자열이거나 공백만 있어서는 안 됩니다.
- 공백을 포함해 최대 100자여야 합니다.
이 조건 중 하나라도 충족되지 않으면 상태 코드 400으로 응답하고 title이라는 단어가 포함된 error 문장을 반환하세요. 예를 들면 다음과 같습니다.
```
{ "error": "title must be a string of 1 to 100 characters" }
```
PATCH 라우트에서는 요청 본문에 포함된 필드만 확인하세요:
- title이 포함되어 있다면 동일한 규칙을 적용하고 같은 종류의 오류를 반환하세요.
- done이 포함되어 있다면 true 또는 false여야 합니다. 그렇지 않으면 상태 코드 400으로 응답하고 done이라는 단어가 포함된 오류를 반환하세요.
```
{ "error": "done must be true or false" }
```
거부된 PATCH 요청은 아무것도 변경하지 않습니다.
올바른 요청 본문은 계속 정상적으로 작동해야 합니다. POST는 여전히 201을 반환하고, { "done": true }를 사용한 PATCH는 여전히 200을 반환합니다. PUT은 그대로 두세요. 스키마도 zod로 확인합니다.

테스트해 보세요: /api/tasks에 { "title": 123 }을 POST하면 제목 오류와 함께 400을 받습니다. { "title": " " }를 POST해도 400을 받습니다. { "done": "yes" }를 사용해 PATCH /api/tasks/2를 요청하면 done 오류와 함께 400을 받고, GET /api/tasks/2의 결과에는 여전히 "done": false가 표시됩니다. { "done": true }를 사용해 PATCH /api/tasks/2를 요청하면 200을 받습니다.

직접 해보기

import { Router } from 'express';
import { nextId, tasks } from '../store.js';

const router = Router();

router.get('/', (req, res) => {
    res.json(tasks);
});

router.get('/:id', (req, res) => {
    const task = tasks.find((t) => t.id === Number(req.params.id));
    if (!task) {
        return res.status(404).json({ error: 'Task not found' });
    }
    res.json(task);
});

router.post('/', (req, res) => {
    const { title } = req.body ?? {};
    if (!title) {
        return res.status(400).json({ error: 'title is required' });
    }
    const task = { id: nextId(), title, done: false, createdAt: new Date().toISOString() };
    tasks.push(task);
    res.status(201).location(`/api/tasks/${task.id}`).json(task);
});

router.patch('/:id', (req, res) => {
    const task = tasks.find((t) => t.id === Number(req.params.id));
    if (!task) {
        return res.status(404).json({ error: 'Task not found' });
    }
    const { title, done } = req.body ?? {};
    if (title !== undefined) task.title = title;
    if (done !== undefined) task.done = done;
    res.json(task);
});

router.put('/:id', (req, res) => {
    const index = tasks.findIndex((t) => t.id === Number(req.params.id));
    if (index === -1) {
        return res.status(404).json({ error: 'Task not found' });
    }
    const { title, done } = req.body ?? {};
    if (title === undefined || done === undefined) {
        return res.status(400).json({ error: 'PUT needs every field: title and done' });
    }
    const { id, createdAt } = tasks[index];
    tasks[index] = { id, title, done, createdAt };
    res.json(tasks[index]);
});

router.delete('/:id', (req, res) => {
    const index = tasks.findIndex((t) => t.id === Number(req.params.id));
    if (index === -1) {
        return res.status(404).json({ error: 'Task not found' });
    }
    tasks.splice(index, 1);
    res.status(204).end();
});

export default router;