diff --git a/README.md b/README.md index 78b1e1d..87cec43 100644 --- a/README.md +++ b/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 diff --git a/lib/walletutils.js b/lib/walletutils.js index 092e3ec..5829c82 100644 --- a/lib/walletutils.js +++ b/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('-'); diff --git a/test/integration/clientApi.js b/test/integration/clientApi.js index 8782b3f..43760e5 100644 --- a/test/integration/clientApi.js +++ b/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(); diff --git a/test/walletutils.js b/test/walletutils.js index 192df87..b3e187e 100644 --- a/test/walletutils.js +++ b/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); + }; + }); + }); + });