Lots of if-elses

[morkeltry]

in router.js.
The alternative would be to first test for existence or non-existence in a whitelist, eg as an array.
Then a case statement would be neater than if-elses.
Even more elegant than case would be designing the whitelist array keyed by route, including actions to take,
eg

whitelistedActions=   {
  '/' : handlers.homeHandler,
  '/home': handler.everyoneGoHome,
  '/loadcomments': getComments
}
...
whitelistedActions[endpoint] (req, res, endpoint);

or

whitelist= {
 '/': ['index.html','text/html'],
 'favicon.ico': ['/images/favicon.ico','image/png'],
 'script.js': ['/views/frontendscript.js','application/javascript']
};
...
staticFileHandler (req,res, whitelist [endpoint]);

or, in your case, a combination of parts of the two with logic..

}

[shannonjensen]

ooh, thank you for the suggestion...