Improves key matching algorithm

Author: flovilmart

src/middlewares.js

@@ -99,20 +99,21 @@ function handleParseHeaders(req, res, next) {
 
   // Client keys are not required in parse-server, but if any have been configured in the server, validate them
   //  to preserve original behavior.
-  var keyRequired = (req.config.clientKey
-    || req.config.javascriptKey
-    || req.config.dotNetKey
-    || req.config.restAPIKey);
-  var keyHandled = false;
-  if (keyRequired
-    && ((info.clientKey && req.config.clientKey && info.clientKey === req.config.clientKey)
-      || (info.javascriptKey && req.config.javascriptKey && info.javascriptKey === req.config.javascriptKey)
-      || (info.dotNetKey && req.config.dotNetKey && info.dotNetKey === req.config.dotNetKey)
-      || (info.restAPIKey && req.config.restAPIKey && info.restAPIKey === req.config.restAPIKey)
-    )) {
-    keyHandled = true;
-  }
-  if (keyRequired && !keyHandled) {
+  var keyMismatch = ["clientKey", "javascriptKey", "dotNetKey", "restAPIKey"].reduce(function(mismatch, key){
+    let configKey = req.config[key];
+    // No key provided in the config, skip
+    if (!configKey) {
+      return mismatch;
+    }
+    // we haven't found a matching key
+    if (mismatch) {
+      // check the info key
+      mismatch = info[key] !== configKey;
+    }
+    return mismatch;
+  }, true);
+  
+  if (keyMismatch) {
     return invalidRequest(req, res);
   }