🤔 Node.js Response Error

😵 Error

상황

POST /users/:userId/orders 에 요청을 보냈을 때, 정상적인 요청이라면 500or 201으로 응답되어야 하고,
정상적인 요청이 아니라면 400으로 응답되어야 한다.

구현 코드

    post: (req, res) => {
      const userId = req.params.userId;
      const { orders, totalPrice } = req.body;

      if (!!orders && !!totalPrice) {
        models.orders.post(userId, orders, totalPrice, (error, result) => {
          if (error) {
            res.status(500).send('Internal Server Error');
          } else {
            res.sendStatus(201);
          }
        });
      }
         res.status(400).json('badbad');
    },

실행

문제

문제 발생은 다음과 같다.

1. 정상적인 요청으로 보내었을 떄 response가 400이 발생한다.

---

🤔 문제 분석

에러 메시지

Error [ERR_HTTP_HEADERS_SENT]: Cannot set headers after they are sent to the client

나의 착각

한번의 response가 실행되면 된다면 뒤의 코드는 실행되지 않는다고 생각하였다. 즉, return 해준 것과 같이, 함수가 끝나버린다고 생각하였다.

사실파악

위와 같이 쓴다면 res.sendStatus(201)와 res.sendStatus(400)모두 실행된다.
즉, 두번의 response가 보내지고, 에러가 발생하게 된다.

간단한 확인을 위해 다음을 실행해 본다면

import express from 'express';

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

app.get('/users/:id', (req, res, next) => {
  let id = req.params.id;
  if (id === 'a') {
    console.log(`before sending 200 status`);
    res.status(200).send('success');
  }
  console.log('after sending 200 status , id', id);
  res.status(404).send('failed');
});

app.listen(8080);

GET /users/1 요청을 보내면 다음과 같은 메시지를 받게 된다.
before sending 200 status
after sending 200 status , id a

Error [ERR_HTTP_HEADERS_SENT]: Cannot set headers after they are sent to the client

if문 안의 console과 if문 밖의 console 모두가 실행되었음을 확인할 수 있다.

에러메시지 : 클라이언트에게 보낸 후 다시 headers를 set할 수 없다.

즉, response가 이루어졌다고 해서 미들웨어 함수가 끝나는 것이 아니다. 이후에 코드가 남아있다면 코드가 이어서 실행된다.

---

😎 개선

 post: (req, res) => {
      const userId = req.params.userId;
      const { orders, totalPrice } = req.body;

      if (!!orders && !!totalPrice) {
        models.orders.post(userId, orders, totalPrice, (error, result) => {
          if (error) {
            res.status(500).send('Internal Server Error');
          } else {
            res.sendStatus(201);
          }
        });
      } else {
        res.sendStatus(400);
      }
    }

if else문으로 분기를 해주거나 return문을 사용하여 한번의 response만 보내지도록 해야한다.

실행

정상적으로 응답을 받아온다.