在module.exports中使用需要模块变量的函数

Question

So I am trying to avoid requiring more than once and so I am designing all of my modules to take arguments from the main app.js file. I have a module for users to login that looks like this:

module.exports = (express, router, jwt, user, config) => {
  function jwtSignUser (user) {
    const ONE_WEEK = 60 * 60 * 24 * 7
    return jwt.sign(user, config.authentication,jwtSecret, {
      expiresIn: ONE_WEEK
    })
  }
  router.post('/login', function(req, res, next) {
    const result = user.findOne(
      {
        email: req.body.email   
      },
      function (err, user) {
        if (err) {
          res.status(400).send(err)
        }
        console.log(user)
        res.send({
          user: user,
          token: jwtSignUser(user)
        })
      }
    )
  })
  return router
}

But when I try to run it I get an error TypeError: jwtSignUser is not a function

How can I use this function but still have the module take in the arguments necessary for the function (like jwt)? Is there a better way I can structure this? I am a newbie to Javascript

Answer 1

I don't know if this is the "best" way to do this. But I did some re-factoring and got it to work by doing the following:

var jwtSignUser = (jwt, config, user) => {
  const ONE_WEEK = 60 * 60 * 24 * 7
  return jwt.sign(user, config.authentication.jwtSecret, {
    expiresIn: ONE_WEEK
  })
}

var main = (express, router, jwt, user, config) => {
  router.post('/login', function(req, res, next) {
    const result = user.findOne(
      {
        email: req.body.email   
      },
      function (err, user) {
        if (err) {
          res.status(400).send(err)
        }
        console.log(user)
        res.send({
          user: user,
          token: jwtSignUser(jwt, config, user.toObject())
        })
      }
    )
  })
  return router
}

module.exports = main

Update

This is not the true reason for why this code now works. There is a part where I have config.authentication,jwtSecret which should be config.authentication.jwtSecret . It now seems to work regardless of where I stick the jstSignUser function