Как отправить ошибку на клиент в koa?

Question

[Wasya UK]

На сервере все ок, вроде. Но на клиент приходит 404 вместо 403. Что я делаю не так?

Нода:

router.post('/api/auth/sign-up', async (ctx, next) => {
  const user = new UserModel({
    ['info.name']: ctx.request.body.username,
    email: ctx.request.body.email,
    password: ctx.request.body.password
  });

  user
    .save()
    .then(user => {
      ctx.body = user;
    })
    .catch(err => {
      ctx.throw(403, "Cannot create user or user is already created!");
    });

  await next();
});

app.use(async (ctx, next) => {
  try {
    await next();
  } catch (err) {
    ctx.status = err.statusCode || err.status || 500;
    ctx.body = {code: err.statusCode, message: err.message};
    ctx.app.emit('error', err, ctx);
  }
})

app.on('error', (err, ctx) => {
  console.error(err);
});

app.use(router.routes());

Реакт:

handleSubmit = e => {
  e.preventDefault();

  if (this.validateForm() === true) {
    axios
      .post('/api/auth/sign-up', {
        username: this.state.username,
        email: this.state.email,
        password: this.state.password
      })
      .then(response => {
        console.log(response);
      })
      .catch(err => {
        console.error(err);
      });
  } else {
    alert('incorrect data');
  }
}

Comments

[hzzzzl]

router.post('/api/auth/sign-up', async (ctx, next) => { })
до функции с роутом точно доходит запрос? 404 больше похоже на то что роут не найден

[Wasya UK]

hzzzzl, наверное да, но почему? Тут, ведь в случае ошибки сообщение должно отправляться пользователю, но его нет(

ctx.throw(403, "Cannot create user or user is already created!"); // создаю
ctx.body = {code: err.statusCode, message: err.message}; // отправляю

Answer 1

[Ильнур Ибаутллин]

1) в POST методе не нужен в конце `await next()`
в принципе, когда обрабатываете конечные пути, `next` не нужен, он нужен только для middleware, когда надо запрос пустить дальше.
2) раз начали работать с async/await - работайте только с ними, не используйте promise

try {
  await user.save()
  ctx.body = user;
} catch (err) {
  ctx.throw(403, "Cannot create user or user is already created!");
}

Answer 2

[Антон Спирин]

router.post('/api/auth/sign-up', ctx => {
  const { username, email, password } = ctx.request.body;

  const user = new UserModel({
    'info.name': username,
    email,
    password,
  });

  user
    .save()
    .then(user => {
      ctx.body = user;
    })
    .catch(err => {
      ctx.throw(403, "Cannot create user or user is already created!");
    });
});

Answer 3

[Виталий]

Вероятно проблема в том что ты не знаешь js , когда ты используешь промисы в асинк функциях результат их исполнения будет существенно позже await next... И не будет важен уже для ответа клиенту.

Answer 4

[Wasya UK]

router.post('/api/auth/sign-up', async (ctx, next) => {
  const user = new UserModel({
    ['info.name']: ctx.request.body.username,
    email: ctx.request.body.email,
    password: ctx.request.body.password
  });

  await user
    .save()
    .catch(err => {
      ctx.throw(403, "Cannot create user or user is already created!");
    });

  ctx.body = user;
  await next();
});

Comments

[Виталий]

ты уверен что вот такой код стоит отмечать правильным решением для потомков?

[Wasya UK]

Виталий, а в чем проблема?