Merge pull request #51 from keen/apryka-params-validation

add basic validation for client

lib/index.js

@@ -3,6 +3,8 @@ import KeenLibrary from 'keen-core';
 
 import each from 'keen-core/lib/utils/each';
 import extend from 'keen-core/lib/utils/extend';
+
+import { validateAuthCredentials } from './utils/validate-auth-credentials';
 import { mapKeysToUnderscore } from './utils/keys-to-underscore';
 
 import pkg from '../package.json';
@@ -14,6 +16,7 @@ KeenLibrary.prototype.readKey = function(str){
 };
 
 KeenLibrary.prototype.query = function(a, b = undefined, options = {}){
+  validateAuthCredentials(this.config);
   // a - analysis type or config object
   // b - params
   const mapObj = {

lib/request.js

@@ -67,6 +67,7 @@ request.prototype.timeout = function(num){
 };
 
 request.prototype.send = function(obj){
+  if (this.config && !this.config.api_key) throw new Error('Please provide valid API key');
   if (obj) {
     this.config.params = (obj && typeof obj === 'object') ? mapKeysToUnderscore(obj) : {};
   }

lib/utils/validate-auth-credentials.js

@@ -0,0 +1,6 @@
+export function validateAuthCredentials(config) {
+    if (typeof config !== 'object') return;
+    if (!config.projectId) throw new Error('Please provide valid project ID');
+    if (!config.masterKey && !config.readKey) throw new Error('Please provide valid API key');
+    return true;
+}