add padding to walletId base58

README.md

@@ -19,14 +19,14 @@ A Multisig HD Wallet Service, with minimun server trust.
  # Create a 2-of-2 wallet (john.dat is the file where the wallet critical data will be stored, add -t for testnet)
  ./bit  create 2-2 john 
   * Secret to share:
-    XYbdNQjWgoTr8uzGMcmkdqKy4fVLGePvfVXQHb8YZM2RJKfvf2pcE8KzTZtxHQNVFM2aJYsFv3T
+    JevjEwaaxW6gdAZjqgWcimL525DR8zQsAXf4cscWDa8u1qKTN5eFGSFssuSvT1WySu4YYLYMUPT
  ./bit  status
  
  # Use -h or BIT_HOST to setup the base URL for your server.
  # Use -f or BIT_FILE to setup the wallet data file
  
  # Join the wallet from other copayer
-   ./bit -f pete.dat join XYbdNQjWgoTr8uzGMcmkdqKy4fVLGePvfVXQHb8YZM2RJKfvf2pcE8KzTZtxHQNVFM2aJYsFv3T
+   ./bit -f pete.dat join JevjEwaaxW6gdAZjqgWcimL525DR8zQsAXf4cscWDa8u1qKTN5eFGSFssuSvT1WySu4YYLYMUPT
    
    export BIT_FILE=pete.dat
    ./bit -f pete.dat status

lib/walletutils.js

@@ -1,3 +1,5 @@
+'use strict';
+
 var _ = require('lodash');
 var $ = require('preconditions').singleton();
 var sjcl = require('sjcl');
@@ -86,8 +88,8 @@ WalletUtils.xPubToCopayerId = function(xpub) {
 
 WalletUtils.toSecret = function(walletId, walletPrivKey, network) {
   var widHex = new Buffer(walletId.replace(/-/g, ''), 'hex');
-  widBase58 = new encoding.Base58(widHex).toString();
-  return widBase58 + walletPrivKey.toWIF() + (network == 'testnet' ? 'T' : 'L');
+  var widBase58 = new encoding.Base58(widHex).toString();
+  return _.padRight(widBase58, 22, '0') + walletPrivKey.toWIF() + (network == 'testnet' ? 'T' : 'L');
 };
 
 WalletUtils.fromSecret = function(secret) {
@@ -106,7 +108,7 @@ WalletUtils.fromSecret = function(secret) {
 
   try {
     var secretSplit = split(secret, [22, 74]);
-    var widBase58 = secretSplit[0];
+    var widBase58 = secretSplit[0].replace(/0/g, '');
     var widHex = encoding.Base58.decode(widBase58).toString('hex');
     var walletId = split(widHex, [8, 12, 16, 20]).join('-');
 

test/integration/clientApi.js

@@ -194,11 +194,11 @@ describe('client API ', function() {
       });
     });
     it('should fail with an invalid secret', function(done) {
-      // Invalid length
+      // Invalid
       clients[0].joinWallet('dummy', 'copayer', function(err, result) {
         err.message.should.contain('Invalid secret');
         // Right length, invalid char for base 58
-        clients[0].joinWallet('lPU9zqUiQFRnt3sWBQ1pnPKx9xC3KD83jxhYVJbLABxn1qhdBfQ7dYtzQqNKpSrDWDqF261S1zT', 'copayer', function(err, result) {
+        clients[0].joinWallet('DsZbqNQQ9LrTKU8EknR7gFKyCQMPg2UUHNPZ1BzM5EbJwjRZaUNBfNtdWLluuFc0f7f7sTCkh7T', 'copayer', function(err, result) {
           err.message.should.contain('Invalid secret');
           done();
         });
@@ -206,7 +206,7 @@ describe('client API ', function() {
     });
     it('should fail with an unknown secret', function(done) {
       // Unknown walletId
-      var oldSecret = 'VPU9zqUiQFRnt3sWBQ1pnPKx9xC3KD83jxhYVJbLABxn1qhdBfQ7dYtzQqNKpSrDWDqF261S1zT';
+      var oldSecret = '3bJKRn1HkQTpwhVaJMaJ22KwsjN24ML9uKfkSrP7iDuq91vSsTEygfGMMpo6kWLp1pXG9wZSKcT';
       clients[0].joinWallet(oldSecret, 'copayer', function(err, result) {
         err.code.should.contain('BADREQUEST');
         done();

test/walletutils.js

@@ -1,9 +1,11 @@
 'use strict';
 
 var _ = require('lodash');
+var Uuid = require('uuid');
 var chai = require('chai');
 var sinon = require('sinon');
 var should = chai.should();
+var Bitcore = require('bitcore');
 var WalletUtils = require('../lib/walletutils');
 
 var aText = 'hola';
@@ -16,19 +18,19 @@ var otherPubKey = '02555a2d45e309c00cc8c5090b6ec533c6880ab2d3bc970b3943def989b33
 describe('WalletUtils', function() {
 
   describe('#hashMessage', function() {
-    it('Should create a hash', function() {
+    it('should create a hash', function() {
       var res = WalletUtils.hashMessage(aText);
       res.toString('hex').should.equal('4102b8a140ec642feaa1c645345f714bc7132d4fd2f7f6202db8db305a96172f');
     });
   });
 
   describe('#signMessage', function() {
-    it('Should sign a message', function() {
+    it('should sign a message', function() {
       var sig = WalletUtils.signMessage(aText, aPrivKey);
       should.exist(sig);
       sig.should.equal(aSignature);
     });
-    it('Should fail to sign with wrong args', function() {
+    it('should fail to sign with wrong args', function() {
       (function() {
         WalletUtils.signMessage(aText, aPubKey);
       }).should.throw('Number');
@@ -36,22 +38,22 @@ describe('WalletUtils', function() {
   });
 
   describe('#verifyMessage', function() {
-    it('Should fail to verify a malformed signature', function() {
+    it('should fail to verify a malformed signature', function() {
       var res = WalletUtils.verifyMessage(aText, 'badsignature', otherPubKey);
       should.exist(res);
       res.should.equal(false);
     });
-    it('Should fail to verify a null signature', function() {
+    it('should fail to verify a null signature', function() {
       var res = WalletUtils.verifyMessage(aText, null, otherPubKey);
       should.exist(res);
       res.should.equal(false);
     });
-    it('Should fail to verify with wrong pubkey', function() {
+    it('should fail to verify with wrong pubkey', function() {
       var res = WalletUtils.verifyMessage(aText, aSignature, otherPubKey);
       should.exist(res);
       res.should.equal(false);
     });
-    it('Should verify', function() {
+    it('should verify', function() {
       var res = WalletUtils.verifyMessage(aText, aSignature, aPubKey);
       should.exist(res);
       res.should.equal(true);
@@ -59,7 +61,7 @@ describe('WalletUtils', function() {
   });
 
   describe('#signMessage #verifyMessage round trip', function() {
-    it('Should sign and verify', function() {
+    it('should sign and verify', function() {
       var aLongerText = "Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur. Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum.";
       var sig = WalletUtils.signMessage(aLongerText, aPrivKey);
       WalletUtils.verifyMessage(aLongerText, sig, aPubKey).should.equal(true);
@@ -74,4 +76,21 @@ describe('WalletUtils', function() {
       msg.should.equal('hello world');
     });
   });
+
+  describe('#toSecret #fromSecret round trip', function() {
+    it('should create secret and parse secret', function() {
+      var i = 0;
+      while (i++ < 100) {
+        var walletId = Uuid.v4();
+        var walletPrivKey = new Bitcore.PrivateKey();
+        var network = 'testnet';
+        var secret = WalletUtils.toSecret(walletId, walletPrivKey, network);
+        var result = WalletUtils.fromSecret(secret);
+        result.walletId.should.equal(walletId);
+        result.walletPrivKey.toString().should.equal(walletPrivKey.toString());
+        result.network.should.equal(network);
+      };
+    });
+  });
+
 });