Refactoring

browser.js

@@ -14,13 +14,14 @@ var EC = require("elliptic").ec;
 
 var ec = new EC("secp256k1");
 
-exports.getPublic = function(privateKey) {
+function assert(condition, message) {
-  // `elliptic` doesn't have such checkings so we do it by ourself. We
+  if (!condition) {
-  // should always ensure that library user doesn't try to do something
+    throw new Error(message || "Assertion failed");
-  // dumb.
-  if (privateKey.length !== 32) {
-    throw new Error("Bad private key");
   }
+}
+
+exports.getPublic = function(privateKey) {
+  assert(privateKey.length === 32, "Bad private key");
   // XXX(Kagami): `elliptic.utils.encode` returns array for every
   // encoding except `hex`.
   return new Buffer(ec.keyPair(privateKey).getPublic("arr"));
@@ -28,20 +29,14 @@ exports.getPublic = function(privateKey) {
 
 exports.sign = function(privateKey, msg) {
   var key = ec.keyPair(privateKey);
-  var sig;
+  return new Promise(function(resolve) {
-  try {
+    resolve(new Buffer(key.sign(msg).toDER()));
-    sig = new Buffer(key.sign(msg).toDER());
+  });
-    return Promise.resolve(sig);
-  } catch(e) {
-    return Promise.reject();
-  }
 };
 
 exports.verify = function(key, msg, sig) {
   key = ec.keyPair(key);
-  if (key.verify(msg, sig)) {
+  return new Promise(function(resolve, reject) {
-    return Promise.resolve();
+    return key.verify(msg, sig) ? resolve() : reject();
-  } else {
+  });
-    return Promise.reject();
-  }
 };

index.js

@@ -25,20 +25,9 @@ var getPublic = exports.getPublic = secp256k1.createPublicKey;
  * @return {Promise.<Buffer>} A promise that resolves with the
  * signature and rejects on bad key or message.
  */
-// FIXME(Kagami): What to do in case of invalid nonce?
 exports.sign = function(privateKey, msg) {
-  return new promise(function(resolve, reject) {
+  return new promise(function(resolve) {
-    try {
+    resolve(secp256k1.sign(privateKey, msg));
-      secp256k1.sign(privateKey, msg, function(code, sig) {
-        if (code === 1) {
-          resolve(sig);
-        } else {
-          reject();
-        }
-      });
-    } catch(e) {
-      reject();
-    }
   });
 };
 
@@ -53,12 +42,6 @@ exports.sign = function(privateKey, msg) {
 exports.verify = function(key, msg, sig) {
   var publicKey = key.length === 32 ? getPublic(key) : key;
   return new promise(function(resolve, reject) {
-    secp256k1.verify(publicKey, msg, sig, function(code) {
+    return secp256k1.verify(publicKey, msg, sig) === 1 ? resolve() : reject();
-      if (code === 1) {
-        resolve();
-      } else {
-        reject();
-      }
-    });
   });
 };