Merge pull request #217 from algorand/release/v1.7.2

Release/v1.7.2

CHANGELOG.md

@@ -1,3 +1,9 @@
+# 1.7.2
+## Fixed
+- Fixed msgpack endpoints returning undefined in browsers (#210 and #215).
+- Removed use of class properties (#213).
+## Changed
+- Remove unneeded dependency js-yaml and changed mock-http-server to a dev dependency (#214 and #212).
 # 1.7.1
 ## Fixed
 - Fixed set Accept on GET calls

package.json

@@ -1,6 +1,6 @@
 {
   "name": "algosdk",
-  "version": "1.7.1",
+  "version": "1.7.2",
   "description": "algosdk is Algorand's official javascript SDK",
   "main": "index.js",
   "directories": {
@@ -15,15 +15,12 @@
     "hi-base32": "^0.5.0",
     "js-sha256": "^0.9.0",
     "js-sha512": "^0.8.0",
-    "js-yaml": ">=3.13.1",
     "keccak256": "^1.0.0",
-    "mock-http-server": "^1.4.2",
     "superagent": "^4.1.0",
     "tweetnacl": "^1.0.1"
   },
   "devDependencies": {
     "@babel/core": "^7.6.0",
-    "@babel/plugin-proposal-class-properties": "^7.10.4",
     "@babel/preset-env": "^7.6.0",
     "babel-core": "^6.26.3",
     "babel-minify": "^0.5.1",
@@ -34,6 +31,7 @@
     "cucumber": "^5.1.0",
     "mocha": "^5.2.0",
     "mocha-lcov-reporter": "^1.3.0",
+    "mock-http-server": "^1.4.2",
     "uglify-js": "^3.6.0",
     "watchify": "^3.11.1",
     "xmlhttprequest": "^1.8.0"
@@ -44,7 +42,7 @@
   ],
   "scripts": {
     "test": "mocha -R spec tests/",
-    "build": "browserify --transform [babelify --plugins [ @babel/plugin-proposal-class-properties ] ] src/main.js -s algosdk | minify --simplify --mangle.keepClassName --deadcode.keepFnArgs --comments false --outFile dist/algosdk.min.js"
+    "build": "browserify src/main.js -s algosdk | minify --simplify --mangle.keepClassName --deadcode.keepFnArgs --comments false --outFile dist/algosdk.min.js"
   },
   "author": "Algorand, llc",
   "license": "MIT"

src/client/client.js

@@ -45,13 +45,20 @@ function HTTPClient(token, baseServer, port, headers={}) {
 
     this.get = async function (path, query, requestHeaders={}) {
         try {
-            return await request
+            const format = getAccceptFormat(query);
+            let r = request
                 .get(this.address + path)
                 .set(this.token)
                 .set(this.defaultHeaders)
                 .set(requestHeaders)
-                .set('Accept', getAccceptFormat(query))
+                .set('Accept', format)
                 .query(removeEmpty(query));
+
+            if (format === 'application/msgpack') {
+                r = r.responseType('arraybuffer');
+            }
+
+            return await r;
         } catch (e) {
             throw e;
         }

src/client/v2/algod/block.js

@@ -15,7 +15,7 @@ class Block {
 	 */
 	async do(headers={}) {
 		let res = await this.c.get("/v2/blocks/" + this.round, this.query, headers);
-		if (res.body) {
+		if (res.body && res.body.byteLength > 0) {
 			return encoding.decode(res.body);
 		}
 		return undefined;