Making redirect work with auth logins about hackathon-starter

[sahat]

This is a good approach. The first problem that I ran into was redirectTo being set to:

/fonts/fontawesome-webfont.woff

I was able to get around it by modifying the regex:

if (/^(auth|login|logout|signup|fonts)$/.test(path)) return next();

But this is not the best solution. What if someone adds a font folder, or what about brand logo image that is fetched on each request? I am not very good with regular expressions, could you modify that code so it ignores anything with a file extension. Perhaps there are other edge cases I am missing?

[ammit]

Use this middleware after express.static,

[ammit]

@sahat If there's no problem moving express.static right on the top after app.set('view engine')
then we can use this middleware after passport.session. I got it working on my local setup. What do you think ?

[sahat]

I have placed the middleware between static and router, it seems to work fine. Will need to run more tests to see if there are side-effects of this new change before pushing to master.

app.use(express.static(path.join(__dirname, 'public'), { maxAge: week }));
app.use(function(req, res, next) {
  if (req.method !== 'GET') return next();
  var path = req.path.split('/')[1];
  if (/(auth|login|logout|signup)$/.test(path)) return next();
  req.session.redirectTo = req.path;
  next();
});
app.use(app.router);

[sahat]

Thanks @ammit. I have pushed this change to master branch. 208dd79