Merge pull request #6 from Icehunter/master

uncaught issue would crash server on authorized route

Author: Anthony Bouch

lib/index.js

@@ -36,9 +36,13 @@ internals.implementation = function (server, options) {
         return reply(Boom.unauthorized(null, 'Signature'));
       }
 
-      var parsedSignature = HttpSignature.parseRequest(req);
-      if (!parsedSignature) {
-        return reply(Boom.unauthorized('HTTP authentication header missing signature', 'Signature'));
+      try {
+        var parsedSignature = HttpSignature.parseRequest(req);
+        if (!parsedSignature) {
+          return reply(Boom.unauthorized('HTTP authentication header missing signature'));
+        }
+      } catch (e) {
+          return reply(Boom.badRequest(e, 'Signature'));
       }
 
       settings.validateFunc(request, parsedSignature, function (err, isValid, credentials) {
@@ -82,4 +86,4 @@ internals.implementation = function (server, options) {
   };
 
   return scheme;
-};
+};

package.json

@@ -1,7 +1,7 @@
 {
   "name": "hapi-auth-signature",
   "description": "Signature authentication plugin that wraps https://github.com/joyent/node-http-signature",
-  "version": "2.1.1",
+  "version": "2.1.2",
   "author": "Anthony Bouch <tony@58bits.com> (http://www.58bits.com)",
   "repository":
     {