Почему код перестает работать, после выноса в отдельный контроллер?

Question

[heducose]

Не понимаю, почему это код перестает работать, если я пробую его рефакторить и вынести в отдельный файл контроллер?

Работает:

router.get('/demo',
  jwt(
    { secret: process.env.SECRET }),
  function (req, res) {
    User.findById(req.user.sub, function (err, user) {
      if (err) { console.log('ERROR') }
      if (user) {
        res.send({
          eee: req.user,
          hi: 'SECRET test authorization of registered user'
        })
      } else {
        res.send('ERROR')
      }
    })
})

Не работает:

router.post('/demo', Authentication.secret)

exports.secret = function (req, res) {
  jwt(
    { secret: process.env.SECRET }),
  function (req, res) {
    User.findById(req.user.sub, function (err, user) {
      if (err) { console.log('ERROR') }
      if (user) {
        res.send({
          eee: req.user,
          hi: 'SECRET test authorization of registered user'
        })
      } else {
        res.send('ERROR')
      }
    })
  }
}

Answer 1

[Дмитрий Беляев]

во-первых, потому что jwt очевидно возвращает middleware
во-вторых, внутреннюю функцию никто не вызывает
правильно так:

router.post('/demo', jwt({ secret: process.env.SECRET }),Authentication.secret)

exports.secret = function (req, res) {
    User.findById(req.user.sub, function (err, user) {
      // добавлю еще маленько хорошей практики обработки ошибок:
      if (err) {
        console.log('ERROR')
        res.send('ERROR')
        return;
      }
      res.send({
        eee: req.user,
        hi: 'SECRET test authorization of registered user'
      })
    })
}

Comments

[heducose]

Спасибо, так работает! А вот насчет обработки ошибок - if (err) обрабатывает саму ошибку, а else у меня кидает предупреждение, если же юзер не найден. Вы его убрали...

[heducose]

А вообще error handling меня заводит в тупик, особенно если я усложняю код и разбиваю его на части. Не подскажете, как прокачаться в плане error handling?