Loading information about a competition every time the page is loaded
Is this slowing down your app by a significant amount? If not, I wouldn't worry about it. 

Other than that I would consider removing the custom error handling, and rather pass it on to some other more general error handler at the end of your middleware stack (just call next(err)), and maybe make the Not Found error a 404 status code.