Merge pull request #475 from Fdawgs/refactor/server

Fdawgs · Jan 11, 2022 · f3fd622 · f3fd622
2 parents de5380e + 180d8a4
commit f3fd622
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 8 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -74,7 +74,7 @@
     "eslint-plugin-promise": "^6.0.0",
     "eslint-plugin-security": "^1.4.0",
     "eslint-plugin-security-node": "^1.1.0",
-    "faker": "^5.5.3",
+    "faker": "5.5.3",
     "glob": "^7.2.0",
     "husky": "^7.0.4",
     "jest": "^27.4.7",

diff --git a/src/server.js b/src/server.js
@@ -34,13 +34,8 @@ async function plugin(server, config) {
 		.register(flocOff)
 
 		// Use Helmet to set response security headers: https://helmetjs.github.io/
-		.register(helmet, config.helmet);
+		.register(helmet, config.helmet)
 
-	await server
-		// Rate limiting and 429 response handling
-		.register(rateLimit, config.rateLimit);
-
-	server
 		// Utility functions and error handlers
 		.register(sensible, { errorHandler: false })
 
@@ -50,6 +45,11 @@ async function plugin(server, config) {
 		// Process load and 503 response handling
 		.register(underPressure, config.processLoad);
 
+	await server
+		// Rate limiting and 429 response handling
+		.register(rateLimit, config.rateLimit);
+
+	// Register routes
 	server
 		// Ensure rate limit also applies to 4xx and 5xx responses
 		.addHook("onSend", server.rateLimit())