adding check for missing content-type header, fixes #2

lib/RouteSchemaManager.js

@@ -237,6 +237,9 @@ var RouteSchemaManager = function(options) {
 		if (!schemas || !schemas[validationTypes.PAYLOAD]) {
 			return { valid: true };
 		}
+		if (!request.raw.req.headers['content-type']) {
+			return { valid: false, errors: ['unable to validate payload: missing content-type header'] };
+		}
 
 		// convert payload types before validating only if payload is type application/x-www-form-urlencoded
 		if (request.raw.req.headers['content-type'].indexOf('application/x-www-form-urlencoded') === 0) {

package.json

@@ -4,7 +4,7 @@
 	"contributors": [
 		"Mac Angell <mac.ang311@gmail.com>"
 	],
-	"version": "0.5.1",
+	"version": "0.5.2",
 	"dependencies": {
 		"async": "0.x.x",
 		"z-schema": "2.x.x",

test/RouteSchemaManager.tests.js

@@ -482,6 +482,52 @@ describe('RouteSchemaManager Unit Tests', function() {
 				done();
 			});
 		});
+
+		it('should fail validation of payload when no payload is present', function(done) {
+
+			var routeSchemaManager = new RouteSchemaManager(rsmConfig),
+				mockRequest = {
+					_route: mockRoute1,
+					payload: null,
+					raw: {
+						req: {
+							headers: {
+								'content-type': 'application/json'
+							}
+						}
+					}
+				};
+			routeSchemaManager.initializeRoutes(mockRoute1.server.info.uri, mockRoutes, function(error) {
+				assert(!error, 'initialize should not return error');
+				var report = routeSchemaManager.validatePayload(mockRequest);
+				assert(!report.valid, 'payload obj should not be valid');
+				assert(report.errors, 'errors obj should be valid');
+				done();
+			});
+		});
+
+		it('should fail validation of payload when no content-type header present', function(done) {
+
+			var routeSchemaManager = new RouteSchemaManager(rsmConfig),
+				mockRequest = {
+					_route: mockRoute1,
+					payload: {
+						string: 'fnord'
+					},
+					raw: {
+						req: {
+							headers: {}
+						}
+					}
+				};
+			routeSchemaManager.initializeRoutes(mockRoute1.server.info.uri, mockRoutes, function(error) {
+				assert(!error, 'initialize should not return error');
+				var report = routeSchemaManager.validatePayload(mockRequest);
+				assert(!report.valid, 'payload obj should not be valid');
+				assert(report.errors, 'errors obj should be valid');
+				done();
+			});
+		});
 	});
 
 /*

test/integration/ratify.tests.js

@@ -75,7 +75,7 @@ before(function(done) {
 
 
 
-describe('hapi-statsd plugin tests', function() {
+describe('ratify plugin tests', function() {
 
 	it('should provide a swagger docs end point', function(done) {