Fix K8S auth (#5)

Signed-off-by: Nick Lototskiy <[email protected]>
namecheap · Nov 21, 2019 · 7560236 · 7560236
1 parent 1d6ebd4
commit 7560236
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/src/VaultClient.js b/src/VaultClient.js
@@ -147,7 +147,7 @@ class VaultClient {
                     api,
                     this.__log,
                     authConfig.config,
-                    authConfig.mount,
+                    authConfig.mount
                 );
         }
 

diff --git a/src/auth/VaultKubernetesAuth.js b/src/auth/VaultKubernetesAuth.js
@@ -24,7 +24,7 @@ class VaultKubernetesAuth extends VaultBaseAuth {
 
         return readFile(this.__filePath)
             .then((jwt) => {
-                this.__apiClient.makeRequest('POST', `/auth/${this._mount}/login`, {
+                return this.__apiClient.makeRequest('POST', `/auth/${this._mount}/login`, {
                     role: this.__role,
                     jwt,
                 });

diff --git a/src/utils/readFile.js b/src/utils/readFile.js
@@ -2,6 +2,8 @@ const { readFile } = require('fs');
 
 module.exports = function (filePath) {
     return new Promise((resolve, reject) => {
-        readFile(filePath, (err, data) => (err ? reject(err) : resolve(data)));
+        readFile(filePath, (err, data) => {
+            return err ? reject(err) : resolve(data.toString());
+        });
     });
 };