Remove msgpack dependency for tinylib/msgp#152

Author: imbstack

package.json

@@ -1,6 +1,6 @@
 {
   "name": "statsum",
-  "version": "0.4.0",
+  "version": "0.4.1",
   "author": "Jonas Finnemann Jensen <[email protected]>",
   "description": "Client library for statsum",
   "license": "MPL-2.0",
@@ -23,9 +23,6 @@
     "lodash": "^4.5.1",
     "url-join": "^0.0.1"
   },
-  "optionalDependencies": {
-    "msgpack": "^1.0.2"
-  },
   "devDependencies": {
     "babel-compile": "^1.0.0",
     "babel-core": "^6.1.21",

src/send.js

@@ -2,26 +2,10 @@ let assert  = require('assert');
 let urljoin = require('url-join');
 let got     = require('got');
 let debug   = require('debug')('statsum');
-let msgpack = null;
-try {
-  msgpack = require('msgpack');
-} catch (err) {
-  debug('Failed to load msgpack (optional dependency) falling back to json');
-}
 
-// Decide whether to use JSON or msgpack.
-// Note: msgpack is less bandwidth, but most importantly it is faster on the
-//       server. JSON is a tiny bit faster on node, but since we have many
-//       clients sending to one server we should optimize for fast server
-//       processing (the server is most likely to a bottleneck).
 let contentType = 'application/json';
 let serialize   = (data) => JSON.stringify(data);
 let deserialize = (data) => JSON.parse(data.toString('utf8'));
-if (msgpack) {
-  contentType = 'application/msgpack';
-  serialize   = (data) => msgpack.pack(data);
-  deserialize = (data) => msgpack.unpack(data);
-}
 
 /** Send data-point to statsum */
 let sendDataPoints = async (configurer, options, payload) => {

test/statsum_test.js

@@ -6,18 +6,8 @@ suite('statsum', () => {
   let getStream = require('get-stream');
   let _         = require('lodash');
   let debug     = require('debug')('statsum');
-  let msgpack = null;
-  try {
-    msgpack = require('msgpack');
-  } catch (err) {
-    debug('Failed to load msgpack (optional dependency) falling back to json');
-  }
-  let deserialize = (data) => JSON.parse(data.toString('utf8'));
-  if (msgpack) {
-    deserialize = (data) => msgpack.unpack(data);
-  }
-
 
+  let deserialize = (data) => JSON.parse(data.toString('utf8'));
   let server = null;
   let baseUrl = null;
   let payload = null;