fix(routes): accept header handling

Fdawgs · Nov 1, 2022 · e5fbb4b · e5fbb4b
1 parent d2ad55d
commit e5fbb4b
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 3 deletions.
diff --git a/src/routes/admin/healthcheck/index.js b/src/routes/admin/healthcheck/index.js
@@ -3,6 +3,8 @@ const cors = require("@fastify/cors");
 
 const { healthcheckGetSchema } = require("./schema");
 
+const accepts = ["text/plain"];
+
 /**
  * @author Frazer Smith
  * @description Sets routing options for server for healthcheck endpoint.
@@ -28,7 +30,7 @@ async function route(server, options) {
 		onRequest: async (req) => {
 			if (
 				// Catch unsupported Accept header media types
-				!req.accepts().type(healthcheckGetSchema.produces)
+				!req.accepts().type(accepts)
 			) {
 				throw server.httpErrors.notAcceptable();
 			}

diff --git a/src/routes/docs/index.js b/src/routes/docs/index.js
@@ -5,6 +5,8 @@ const staticPlugin = require("@fastify/static");
 
 const { docsGetSchema } = require("./schema");
 
+const accepts = ["text/html"];
+
 /**
  * @author Frazer Smith
  * @description Sets routing options for server.
@@ -39,7 +41,7 @@ async function route(server) {
 		onRequest: async (req) => {
 			if (
 				// Catch unsupported Accept header media types
-				!req.accepts().type(docsGetSchema.produces)
+				!req.accepts().type(accepts)
 			) {
 				throw server.httpErrors.notAcceptable();
 			}

diff --git a/src/routes/docs/openapi/index.js b/src/routes/docs/openapi/index.js
@@ -3,6 +3,8 @@ const cors = require("@fastify/cors");
 
 const { docsOpenapiGetSchema } = require("./schema");
 
+const accepts = docsOpenapiGetSchema.produces;
+
 /**
  * @author Frazer Smith
  * @description Sets routing options for server.
@@ -26,7 +28,7 @@ async function route(server, options) {
 		onRequest: async (req) => {
 			if (
 				// Catch unsupported Accept header media types
-				!req.accepts().type(docsOpenapiGetSchema.produces)
+				!req.accepts().type(accepts)
 			) {
 				throw server.httpErrors.notAcceptable();
 			}