Merge pull request #80 from HussainAliAkbar/bg-roleStringValidation-79

Fix: Issue#79: called the utils.deny function when the validation whe…
nyambati · Jun 23, 2017 · 9a2186b · 9a2186b
2 parents bf87d22 + 5548ae3
commit 9a2186b
Showing 1 changed file with 11 additions and 1 deletion.
diff --git a/lib/nacl.js b/lib/nacl.js
@@ -72,7 +72,17 @@ function authorize(req, res, next) {
 
   let role = helper.getRole(req, res, opt.decodedObjectName, opt.defaultRole);
 
-  if (!_.isString(role) || !role) return;
+  /**
+   * if no role or role not provided as string
+   */
+  if (!_.isString(role) || !role) {
+    return utils.deny(
+      res,
+      404,
+      'REQUIRED: Role should be provided as a string',
+      null
+    );
+  }
 
   /**
    * get resource from the url