Cleaning up sonar issues

Author: rossphelan

lib/mcapi/crypto/crypto.js

@@ -18,21 +18,7 @@ function Crypto(config) {
   this.encryptionCertificate = readPublicCertificate(config.encryptionCertificate);
 
   // Load private key (for decryption)
-  if (config.privateKey) {
-    this.privateKey = loadPrivateKey(config.privateKey);
-  } else if (config.keyStore) {
-    if(config.keyStore.includes(".p12")) {
-      this.privateKey = getPrivateKey12(config.keyStore, config.keyStoreAlias, config.keyStorePassword);
-    }
-    if(config.keyStore.includes(".pem")) {
-      this.privateKey = getPrivateKeyPem(config.keyStore);
-    }
-    if(config.keyStore.includes(".der")) {
-      this.privateKey = getPrivateKeyDer(config.keyStore);
-    }
-  }
-
-
+  this.privateKey = getPrivateKey(config)
 
   this.encoding = config.dataEncoding;
 
@@ -203,6 +189,26 @@ function readPublicCertificate(publicCertificatePath) {
   return forge.pki.certificateFromPem(certificateContent);
 }
 
+/**
+ * @private
+ */
+function getPrivateKey(config) {
+  if (config.privateKey) {
+    return loadPrivateKey(config.privateKey);
+  } else if (config.keyStore) {
+    if (config.keyStore.includes(".p12")) {
+      return getPrivateKey12(config.keyStore, config.keyStoreAlias, config.keyStorePassword);
+    }
+    if (config.keyStore.includes(".pem")) {
+      return getPrivateKeyPem(config.keyStore);
+    }
+    if (config.keyStore.includes(".der")) {
+      return getPrivateKeyDer(config.keyStore);
+    }
+  }
+  return null
+}
+
 /**
  * @private
  */
@@ -240,7 +246,6 @@ function getPrivateKey12(p12Path, alias, password) {
   return keyObj.key;
 }
 
-
 /**
  * @private
  */