CODEWITHSUNDEEP
javascriptnode.jsexpressexpress-router
Booi Mangang
Booi Mangang

Reputation: 31

Route.get() requires a callback function but got a [object Undefined]. What did I do wrong?

I have checked many answers in other pages and forums, but I still don't get it. What did I do wrong? Help me

*edited. I have added requiring routes and app.use. It looks like function isLOggedIn is not exporting but I don't know how to do that. I did like this in another app, it worked there.

auth-routes.js

const express = require("express"),
  router      = express.Router(),
  passport    = require("passport")

function isLoggedIn(req, res, next) {
  if (req.isAuthenticated()) {
    return next()
  }

  res.redirect("/auth/login")
}

module.exports = router

user-routes.js

const express  = require("express"),
  router = express.Router(),
  authRoutes = require("./auth-routes")   

router.get("/profile", authRoutes.isLoggedIn, (req, res) => {
  res.render("user/profile", {user: req.user})
})

module.exports = router

requring routes

const titleRoutes = require("./routes/title-routes")
const authRoutes = require("./routes/auth-routes")
const userRoutes = require("./routes/user-routes")

app.use

app.use(titleRoutes)
app.use("/auth", authRoutes)
app.use("/user", userRoutes)

Upvotes: 3

Views: 7949

Answers (1)

Taymer
Taymer

Reputation: 314

In auth-routes.js you don't export isLoggedIn. So in user-routes.js, authRoutes.isLoggedIn is undefined.

You can change:

module.exports = router

into:

exports.isLoggedIn = isLoggedIn

or using module.exports into:

module.exports = { isLoggedIn: isLoggedIn }

A useful link to understand export in nodejs https://www.sitepoint.com/understanding-module-exports-exports-node-js/

Upvotes: 5

Related Questions