@trenskow/app 0.5.14 → 0.6.0

package/README.md

@@ -499,6 +499,8 @@ You can only call these methods once per method per endpoint – calling it mult
 
 These also ends routing. After a method route has been called, the routing will go strait to the renderer.
 
+> Notice: If no `head` method is implemented on endpoint, `get` will instead be called (if availble). When client requests a `head` the result will be ignored.
+
 > Returns the endpoint.
 
 ###### Parameters

package/lib/endpoint.js

@@ -226,7 +226,7 @@ export default class Endpoint extends Router {
 
 				let underlyingMethod = context.request.method.toLowerCase();
 
-				if (underlyingMethod === 'head') {
+				if (underlyingMethod === 'head' && !this._layers.some((layer) => layer.method === 'head')) {
 					underlyingMethod = 'get';
 				}
 
@@ -235,6 +235,7 @@ export default class Endpoint extends Router {
 
 				for (let handler of layer.handlers) {
 					const result = await handler(context);
+					if (layer.underlyingMethod === 'head') return;
 					if (context.state !== 'routing') return;
 					if (typeof result !== 'undefined') context.result = result;
 				}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@trenskow/app",
-  "version": "0.5.14",
+  "version": "0.6.0",
   "description": "A small HTTP router.",
   "type": "module",
   "main": "index.js",

package/test/index.js

@@ -74,7 +74,24 @@ describe('Application', () => {
 
 		});
 
-		it ('should respond with 200 and no content when a GET method is configured and HEAD is requested.', async () => {
+		it ('should respond with 200 when HEAD method is configured and HEAD method is requested', async () => {
+
+			app.root(
+				new Endpoint()
+					.get(() => 'Hello, World!')
+					.head(({ response }) => {
+						response.statusCode = 204;
+						return 'Hello, World!';
+					})
+			);
+
+			await request
+				.head('/')
+				.expect(204, undefined);
+
+		});
+
+		it ('should respond with 200 and no content when a GET method is configured (but no HEAD method) and HEAD is requested.', async () => {
 
 			app.root(
 				new Endpoint()