Complete refactor of app based on OOP principles

.prettierrc

@@ -1,5 +1,4 @@
 {
 	"tabWidth": 4,
-	"useTabs": true,
-	"printWidth": 150
+	"useTabs": true
 }

README.md

@@ -41,7 +41,7 @@ If you want to contribute to development of Compass, we encourage you to explore
 
 2.  Run the `db_setup` script in the `db` folder to build the database.
 3.  Create `.env` file and insert your secrets into it. Use `.env.example` file as reference for what is required.
-4.  Run `nodemon` and open the app at `localhost:<port>` specifying the port defined in `index.js`.
+4.  Run `nodemon` and open the app at `localhost:<port>` specifying the port defined in `app.js`.
 
         $ npm install -g nodemon
         $ nodemon

app.js

@@ -0,0 +1,6 @@
+"use strict";
+
+require("dotenv").config();
+
+const { Server } = require("./controllers/Server");
+new Server();

controllers/Courses.js

@@ -0,0 +1,26 @@
+"use strict";
+const Router = require("./Router");
+
+class Courses extends Router {
+	constructor(app, passport, db) {
+		super(app, passport, db);
+	}
+
+	setRoutes(db) {
+		this.router.get("/", (req, res) => {
+			db.Course.findAll().then((result) => {
+				console.log(req.user);
+				res.render("index.html", {
+					title: "Compass – Courses",
+					heading: "Courses",
+					courseReviewData: result,
+					user: req.user,
+					errorMessage: req.flash("error"),
+					warningMessage: req.flash("warning"),
+				});
+			});
+		});
+	}
+}
+
+module.exports = Courses;

controllers/Errors.js

@@ -0,0 +1,26 @@
+"use strict";
+
+const Router = require("./Router");
+
+class Errors extends Router {
+	constructor(app, passport, db) {
+		super(app, passport, db);
+	}
+
+	setRoutes(db) {
+		// Handle pages not found.
+		this.app.use((req, res) => {
+			res.status(404).render("404.html", {
+				title: "Compass - Page not found",
+			});
+		});
+
+		// Handle 500 server errors.
+		this.app.use((err, req, res) => {
+			console.error(err.stack);
+			res.status(500).send("<h1>500: Internal server error</h1>");
+		});
+	}
+}
+
+module.exports = Errors;

controllers/HttpServer.js

@@ -0,0 +1,37 @@
+const fs = require("fs");
+const http = require("http");
+const https = require("https");
+
+class HttpServer {
+	constructor(app) {
+		this.createHttpServer(app);
+	}
+
+	createHttpServer(app) {
+		const httpServer = http.createServer(app);
+		const httpPort = process.env.HTTP_PORT || "8087";
+		const httpsPort = process.env.HTTPS_PORT || "8443";
+
+		httpServer.listen(httpPort, () => {
+			console.log(`HTTP Server started at port ${httpPort}`);
+		});
+
+		if (process.env.NODE_ENV === "PROD") {
+			const certDir = process.env.TLS_CERTDIR;
+			const tlsOptions = {
+				key: fs.readFileSync(certDir + "privkey.pem", "utf8"),
+				cert: fs.readFileSync(certDir + "cert.pem", "utf8"),
+				ca: fs.readFileSync(certDir + "chain.pem", "utf8"),
+			};
+
+			// create HTTPS server
+			const httpsServer = https.createServer(tlsOptions, app);
+
+			httpsServer.listen(httpsPort, () => {
+				console.log(`HTTPS Server started at port ${httpsPort}!`);
+			});
+		}
+	}
+}
+
+module.exports = HttpServer;

controllers/Passport.js

@@ -0,0 +1,88 @@
+"use strict";
+
+const passport = require("passport");
+
+class Passport {
+	constructor(app, db) {
+		this.passport = passport;
+		this.createPassportSession(app, db);
+	}
+
+	createPassportSession(app, db) {
+		app.use(passport.initialize(undefined));
+		app.use(passport.session(undefined));
+
+		// use `id` for serializing and deserializing users in database
+		passport.serializeUser(function (user, done) {
+			console.log("Serializing " + user.id);
+			done(null, user.id);
+		});
+
+		passport.deserializeUser(function (id, done) {
+			console.log("Deserializing " + id);
+
+			let result = db.User.findByPk(id)
+				.then(done(null, true))
+				.catch((err) => {
+					done(err, null, { message: "User does not exist" });
+				});
+		});
+
+		const SlackStrategy = require("passport-slack").Strategy;
+
+		passport.use(
+			"slack",
+			new SlackStrategy(
+				{
+					clientID: process.env.SLACK_CLIENT_ID,
+					clientSecret: process.env.SLACK_CLIENT_SECRET,
+					scope: [
+						"identity.basic",
+						"identity.email",
+						"identity.avatar",
+					],
+				},
+				(accessToken, refreshToken, profile, done) => {
+					db.User.findByPk(profile.user.id)
+						.then((user) => {
+							if (!user) {
+								db.User.create({
+									id: profile.user.id,
+									name: profile.user.name,
+									email: profile.user.email,
+									avatar_url: profile.user.image_512,
+								})
+									.then(() => {
+										return done(null, user);
+									})
+									.catch((err) => {
+										console.log(
+											"User couldn't be created: " + err
+										);
+									});
+							}
+
+							return done(null, user);
+						})
+						.catch((err) => {
+							console.log(err);
+						});
+				}
+			)
+		);
+	}
+
+	checkAuth(req, res, next) {
+		if (req.isAuthenticated()) {
+			next();
+		} else {
+			req.flash(
+				"warning",
+				"You have to sign in before you can access this page"
+			);
+			res.redirect("/courses");
+		}
+	}
+}
+
+module.exports = Passport;