Add more protections

index.js

@@ -20,12 +20,17 @@ app.get("/:domain/*", (req, res) => {
 	const domain = req.params.domain;
 	const path = req.params[0];
 	const args = req.query;
-	// Check that domain is equal or subdomain of allowedDomains
+	// If domain has any characters that arent alphanumeric, a period, or -, return 400
-	if (!allowedDomains.some((allowedDomain) => domain.endsWith(allowedDomain))) {
+	if (!/^[a-zA-Z0-9.-]+$/.test(domain)) {
-		res.status(403).send("Domain not allowed");
+		return res.status(400).send("Invalid domain");
-		return;
 	}
 
+	// Check if domain is valid subdomain of allowedDomains
+	if (!allowedDomains.some((allowedDomain) => domain.endsWith(allowedDomain))) {
+		return res.status(400).send("Invalid domain");
+	}
+	
+
 	// Make the request
 	axios.get(`https://${domain}/${path}`, {
 		params: args
@@ -34,10 +39,18 @@ app.get("/:domain/*", (req, res) => {
 		res.status(response.status).send(response.data);
 	}).catch((error) => {
 		// Send the error back
+		if (!error.response) {
+			return res.status(500).send("An error occurred");
+		}
 		res.status(error.response.status).send(error.response.data);
 	});	
 });
 
+app.get("/:domain", (req, res) => {
+	// redirect them to /:domain/
+	res.redirect(`/${req.params.domain}/`);
+})
+
 const port = process.env.SERVER_PORT || 3000;
 app.listen(port, () => {
 	console.log(`Server is running on port ${port}`);