add optional callback to public api and invoke AsyncProxy behind the scenes

This commit is contained in:
Tankred Hase 2014-02-12 14:53:20 +01:00
parent c47e1d8780
commit b0bd5168a8
6 changed files with 185 additions and 66 deletions

1
.gitignore vendored
View File

@ -4,3 +4,4 @@ node_modules/
npm* npm*
test/lib/ test/lib/
dist/ dist/
openpgp.store/

View File

@ -108,7 +108,16 @@ module.exports = function(grunt) {
dest: 'test/lib/' dest: 'test/lib/'
} }
}, },
clean: ['dist/'] clean: ['dist/'],
connect: {
dev: {
options: {
port: 8588,
base: '.',
keepalive: true
}
}
}
}); });
// Load the plugin(s) // Load the plugin(s)
@ -121,6 +130,7 @@ module.exports = function(grunt) {
grunt.loadNpmTasks('grunt-mocha-test'); grunt.loadNpmTasks('grunt-mocha-test');
grunt.loadNpmTasks('grunt-contrib-copy'); grunt.loadNpmTasks('grunt-contrib-copy');
grunt.loadNpmTasks('grunt-contrib-clean'); grunt.loadNpmTasks('grunt-contrib-clean');
grunt.loadNpmTasks('grunt-contrib-connect');
grunt.registerTask('default', 'Build OpenPGP.js', function() { grunt.registerTask('default', 'Build OpenPGP.js', function() {
grunt.task.run(['clean', 'browserify', 'replace', 'uglify', 'npm_pack']); grunt.task.run(['clean', 'browserify', 'replace', 'uglify', 'npm_pack']);

View File

@ -33,6 +33,7 @@
"mocha-phantomjs": "~3.1.6", "mocha-phantomjs": "~3.1.6",
"phantomjs": "~1.9.2-5", "phantomjs": "~1.9.2-5",
"chai": "~1.8.1", "chai": "~1.8.1",
"grunt-contrib-connect": "~0.6.0",
"grunt-contrib-copy": "~0.4.1", "grunt-contrib-copy": "~0.4.1",
"grunt-browserify": "~1.2.11", "grunt-browserify": "~1.2.11",
"grunt-contrib-uglify": "*", "grunt-contrib-uglify": "*",

View File

@ -65,8 +65,3 @@ module.exports.crypto = require('./crypto');
* @name module:openpgp.Keyring * @name module:openpgp.Keyring
*/ */
module.exports.Keyring = require('./keyring'); module.exports.Keyring = require('./keyring');
/**
* @see module:worker/async_proxy
* @name module:openpgp.AsyncProxy
*/
module.exports.AsyncProxy = require('./worker/async_proxy.js');

View File

@ -37,21 +37,40 @@ var armor = require('./encoding/armor.js'),
config = require('./config'), config = require('./config'),
message = require('./message.js'), message = require('./message.js'),
cleartext = require('./cleartext.js'), cleartext = require('./cleartext.js'),
key = require('./key.js'); key = require('./key.js'),
AsyncProxy = require('./worker/async_proxy.js');
var asyncProxy; // instance of the asyncproxy
/**
* Set the path for the web worker script and create an instance of the async proxy
* @param {String} path relative path to the worker scripts
*/
function initWorker(path) {
asyncProxy = new AsyncProxy(path);
}
/** /**
* Encrypts message text with keys * Encrypts message text with keys
* @param {Array<module:key~Key>} keys array of keys, used to encrypt the message * @param {Array<module:key~Key>} keys array of keys, used to encrypt the message
* @param {String} text message as native JavaScript string * @param {String} text message as native JavaScript string
* @param {function} callback (optional) callback(error, result) for async style
* @return {String} encrypted ASCII armored message * @return {String} encrypted ASCII armored message
* @static * @static
*/ */
function encryptMessage(keys, text) { function encryptMessage(keys, text, callback) {
var msg = message.fromText(text); if (useWorker(callback)) {
msg = msg.encrypt(keys); asyncProxy.encryptMessage(keys, text, callback);
var armored = armor.encode(enums.armor.message, msg.packets.write()); return;
return armored; }
return execute(function() {
var msg, armored;
msg = message.fromText(text);
msg = msg.encrypt(keys);
armored = armor.encode(enums.armor.message, msg.packets.write());
return armored;
}, callback);
} }
/** /**
@ -59,80 +78,121 @@ function encryptMessage(keys, text) {
* @param {Array<module:key~Key>} publicKeys array of keys, used to encrypt the message * @param {Array<module:key~Key>} publicKeys array of keys, used to encrypt the message
* @param {module:key~Key} privateKey private key with decrypted secret key data for signing * @param {module:key~Key} privateKey private key with decrypted secret key data for signing
* @param {String} text message as native JavaScript string * @param {String} text message as native JavaScript string
* @param {function} callback (optional) callback(error, result) for async style
* @return {String} encrypted ASCII armored message * @return {String} encrypted ASCII armored message
* @static * @static
*/ */
function signAndEncryptMessage(publicKeys, privateKey, text) { function signAndEncryptMessage(publicKeys, privateKey, text, callback) {
var msg = message.fromText(text); if (useWorker(callback)) {
msg = msg.sign([privateKey]); asyncProxy.signAndEncryptMessage(publicKeys, privateKey, text, callback);
msg = msg.encrypt(publicKeys); return;
var armored = armor.encode(enums.armor.message, msg.packets.write()); }
return armored;
return execute(function() {
var msg, armored;
msg = message.fromText(text);
msg = msg.sign([privateKey]);
msg = msg.encrypt(publicKeys);
armored = armor.encode(enums.armor.message, msg.packets.write());
return armored;
}, callback);
} }
/** /**
* Decrypts message * Decrypts message
* @param {module:key~Key} privateKey private key with decrypted secret key data * @param {module:key~Key} privateKey private key with decrypted secret key data
* @param {module:message~Message} message the message object with the encrypted data * @param {module:message~Message} msg the message object with the encrypted data
* @param {function} callback (optional) callback(error, result) for async style
* @return {(String|null)} decrypted message as as native JavaScript string * @return {(String|null)} decrypted message as as native JavaScript string
* or null if no literal data found * or null if no literal data found
* @static * @static
*/ */
function decryptMessage(privateKey, message) { function decryptMessage(privateKey, msg, callback) {
message = message.decrypt(privateKey); if (useWorker(callback)) {
return message.getText(); asyncProxy.decryptMessage(privateKey, msg, callback);
return;
}
return execute(function() {
msg = msg.decrypt(privateKey);
return msg.getText();
}, callback);
} }
/** /**
* Decrypts message and verifies signatures * Decrypts message and verifies signatures
* @param {module:key~Key} privateKey private key with decrypted secret key data * @param {module:key~Key} privateKey private key with decrypted secret key data
* @param {Array<module:key~Key>} publicKeys public keys to verify signatures * @param {Array<module:key~Key>} publicKeys public keys to verify signatures
* @param {module:message~Message} message the message object with signed and encrypted data * @param {module:message~Message} msg the message object with signed and encrypted data
* @param {function} callback (optional) callback(error, result) for async style
* @return {{text: String, signatures: Array<{keyid: module:type/keyid, valid: Boolean}>}} * @return {{text: String, signatures: Array<{keyid: module:type/keyid, valid: Boolean}>}}
* decrypted message as as native JavaScript string * decrypted message as as native JavaScript string
* with verified signatures or null if no literal data found * with verified signatures or null if no literal data found
* @static * @static
*/ */
function decryptAndVerifyMessage(privateKey, publicKeys, message) { function decryptAndVerifyMessage(privateKey, publicKeys, msg, callback) {
var result = {}; if (useWorker(callback)) {
message = message.decrypt(privateKey); asyncProxy.decryptAndVerifyMessage(privateKey, publicKeys, msg, callback);
result.text = message.getText(); return;
if (result.text) {
result.signatures = message.verify(publicKeys);
return result;
} }
return null;
return execute(function() {
var result = {};
msg = msg.decrypt(privateKey);
result.text = msg.getText();
if (result.text) {
result.signatures = msg.verify(publicKeys);
return result;
}
return null;
}, callback);
} }
/** /**
* Signs a cleartext message * Signs a cleartext message
* @param {Array<module:key~Key>} privateKeys private key with decrypted secret key data to sign cleartext * @param {Array<module:key~Key>} privateKeys private key with decrypted secret key data to sign cleartext
* @param {String} text cleartext * @param {String} text cleartext
* @param {function} callback (optional) callback(error, result) for async style
* @return {String} ASCII armored message * @return {String} ASCII armored message
* @static * @static
*/ */
function signClearMessage(privateKeys, text) { function signClearMessage(privateKeys, text, callback) {
var cleartextMessage = new cleartext.CleartextMessage(text); if (useWorker(callback)) {
cleartextMessage.sign(privateKeys); asyncProxy.signClearMessage(privateKeys, text, callback);
return cleartextMessage.armor(); return;
}
return execute(function() {
var cleartextMessage = new cleartext.CleartextMessage(text);
cleartextMessage.sign(privateKeys);
return cleartextMessage.armor();
}, callback);
} }
/** /**
* Verifies signatures of cleartext signed message * Verifies signatures of cleartext signed message
* @param {Array<module:key~Key>} publicKeys public keys to verify signatures * @param {Array<module:key~Key>} publicKeys public keys to verify signatures
* @param {module:cleartext~CleartextMessage} message cleartext message object with signatures * @param {module:cleartext~CleartextMessage} msg cleartext message object with signatures
* @param {function} callback (optional) callback(error, result) for async style
* @return {{text: String, signatures: Array<{keyid: module:type/keyid, valid: Boolean}>}} * @return {{text: String, signatures: Array<{keyid: module:type/keyid, valid: Boolean}>}}
* cleartext with status of verified signatures * cleartext with status of verified signatures
* @static * @static
*/ */
function verifyClearSignedMessage(publicKeys, message) { function verifyClearSignedMessage(publicKeys, msg, callback) {
var result = {}; if (useWorker(callback)) {
if (!(message instanceof cleartext.CleartextMessage)) { asyncProxy.verifyClearSignedMessage(publicKeys, msg, callback);
throw new Error('Parameter [message] needs to be of type CleartextMessage.'); return;
} }
result.text = message.getText();
result.signatures = message.verify(publicKeys); return execute(function() {
return result; var result = {};
if (!(msg instanceof cleartext.CleartextMessage)) {
throw new Error('Parameter [message] needs to be of type CleartextMessage.');
}
result.text = msg.getText();
result.signatures = msg.verify(publicKeys);
return result;
}, callback);
} }
/** /**
@ -143,18 +203,71 @@ function verifyClearSignedMessage(publicKeys, message) {
* @param {Integer} numBits number of bits for the key creation. (should be 1024+, generally) * @param {Integer} numBits number of bits for the key creation. (should be 1024+, generally)
* @param {String} userId assumes already in form of "User Name <username@email.com>" * @param {String} userId assumes already in form of "User Name <username@email.com>"
* @param {String} passphrase The passphrase used to encrypt the resulting private key * @param {String} passphrase The passphrase used to encrypt the resulting private key
* @param {function} callback (optional) callback(error, result) for async style
* @return {Object} {key: Array<module:key~Key>, privateKeyArmored: Array<String>, publicKeyArmored: Array<String>} * @return {Object} {key: Array<module:key~Key>, privateKeyArmored: Array<String>, publicKeyArmored: Array<String>}
* @static * @static
*/ */
function generateKeyPair(keyType, numBits, userId, passphrase) { function generateKeyPair(keyType, numBits, userId, passphrase, callback) {
var result = {}; if (useWorker(callback)) {
var newKey = key.generate(keyType, numBits, userId, passphrase); asyncProxy.generateKeyPair(keyType, numBits, userId, passphrase, callback);
result.key = newKey; return;
result.privateKeyArmored = newKey.armor(); }
result.publicKeyArmored = newKey.toPublic().armor();
return execute(function() {
var result = {};
var newKey = key.generate(keyType, numBits, userId, passphrase);
result.key = newKey;
result.privateKeyArmored = newKey.armor();
result.publicKeyArmored = newKey.toPublic().armor();
return result;
}, callback);
}
//
// helper functions
//
/**
* Are we in a browser and do we support worker?
*/
function useWorker(callback) {
if (typeof callback === 'undefined') {
return false;
}
if (!asyncProxy) {
throw new Error('You need to set the worker path!');
}
return true;
}
/**
* Command pattern that handles async calls gracefully
*/
function execute(cmd, callback) {
var result;
try {
result = cmd();
} catch (err) {
if (callback) {
callback(err);
return;
}
throw err;
}
if (callback) {
callback(null, result);
return;
}
return result; return result;
} }
exports.initWorker = initWorker;
exports.encryptMessage = encryptMessage; exports.encryptMessage = encryptMessage;
exports.signAndEncryptMessage = signAndEncryptMessage; exports.signAndEncryptMessage = signAndEncryptMessage;
exports.decryptMessage = decryptMessage; exports.decryptMessage = decryptMessage;
@ -162,3 +275,4 @@ exports.decryptAndVerifyMessage = decryptAndVerifyMessage;
exports.signClearMessage = signClearMessage; exports.signClearMessage = signClearMessage;
exports.verifyClearSignedMessage = verifyClearSignedMessage; exports.verifyClearSignedMessage = verifyClearSignedMessage;
exports.generateKeyPair = generateKeyPair; exports.generateKeyPair = generateKeyPair;
exports.AsyncProxy = AsyncProxy;

View File

@ -177,20 +177,17 @@ var priv_key_de =
describe('High level API', function() { describe('High level API', function() {
var proxy;
this.timeout(0); this.timeout(0);
before(function() { before(function() {
proxy = new openpgp.AsyncProxy('../dist/openpgp.worker.js'); openpgp.initWorker('../dist/openpgp.worker.js');
expect(proxy).to.exist;
initKeys(); initKeys();
}); });
describe('Encryption', function() { describe('Encryption', function() {
it('RSA: encryptMessage async', function (done) { it('RSA: encryptMessage async', function (done) {
proxy.encryptMessage([pubKeyRSA], plaintext, function(err, data) { openpgp.encryptMessage([pubKeyRSA], plaintext, function(err, data) {
expect(err).to.not.exist; expect(err).to.not.exist;
expect(data).to.exist; expect(data).to.exist;
expect(data).to.match(/^-----BEGIN PGP MESSAGE/); expect(data).to.match(/^-----BEGIN PGP MESSAGE/);
@ -209,7 +206,7 @@ describe('High level API', function() {
}); });
it('ELG: encryptMessage async', function (done) { it('ELG: encryptMessage async', function (done) {
proxy.encryptMessage([pubKeyDE], plaintext, function(err, data) { openpgp.encryptMessage([pubKeyDE], plaintext, function(err, data) {
expect(err).to.not.exist; expect(err).to.not.exist;
expect(data).to.exist; expect(data).to.exist;
expect(data).to.match(/^-----BEGIN PGP MESSAGE/); expect(data).to.match(/^-----BEGIN PGP MESSAGE/);
@ -241,7 +238,7 @@ describe('High level API', function() {
}); });
it('RSA: decryptMessage async', function (done) { it('RSA: decryptMessage async', function (done) {
proxy.decryptMessage(privKeyRSA, msgRSA, function(err, data) { openpgp.decryptMessage(privKeyRSA, msgRSA, function(err, data) {
expect(err).to.not.exist; expect(err).to.not.exist;
expect(data).to.exist; expect(data).to.exist;
expect(data).to.equal(plaintext); expect(data).to.equal(plaintext);
@ -256,7 +253,7 @@ describe('High level API', function() {
}); });
it('ELG: decryptMessage async', function (done) { it('ELG: decryptMessage async', function (done) {
proxy.decryptMessage(privKeyDE, msgDE, function(err, data) { openpgp.decryptMessage(privKeyDE, msgDE, function(err, data) {
expect(err).to.not.exist; expect(err).to.not.exist;
expect(data).to.exist; expect(data).to.exist;
expect(data).to.equal(plaintext); expect(data).to.equal(plaintext);
@ -291,7 +288,7 @@ describe('High level API', function() {
}); });
it('RSA: decryptAndVerifyMessage async', function (done) { it('RSA: decryptAndVerifyMessage async', function (done) {
proxy.decryptAndVerifyMessage(privKeyRSA, [pubKeyRSA], msgRSA, function(err, data) { openpgp.decryptAndVerifyMessage(privKeyRSA, [pubKeyRSA], msgRSA, function(err, data) {
expect(err).to.not.exist; expect(err).to.not.exist;
expect(data).to.exist; expect(data).to.exist;
verifySignature(data, privKeyRSA); verifySignature(data, privKeyRSA);
@ -300,7 +297,7 @@ describe('High level API', function() {
}); });
it('ELG: decryptAndVerifyMessage async', function (done) { it('ELG: decryptAndVerifyMessage async', function (done) {
proxy.decryptAndVerifyMessage(privKeyDE, [pubKeyDE], msgDE, function(err, data) { openpgp.decryptAndVerifyMessage(privKeyDE, [pubKeyDE], msgDE, function(err, data) {
expect(err).to.not.exist; expect(err).to.not.exist;
expect(data).to.exist; expect(data).to.exist;
verifySignature(data, privKeyDE); verifySignature(data, privKeyDE);
@ -317,7 +314,7 @@ describe('High level API', function() {
}); });
it('RSA: signAndEncryptMessage async', function (done) { it('RSA: signAndEncryptMessage async', function (done) {
proxy.signAndEncryptMessage([pubKeyRSA], privKeyRSA, plaintext, function(err, data) { openpgp.signAndEncryptMessage([pubKeyRSA], privKeyRSA, plaintext, function(err, data) {
expect(err).to.not.exist; expect(err).to.not.exist;
expect(data).to.exist; expect(data).to.exist;
expect(data).to.match(/^-----BEGIN PGP MESSAGE/); expect(data).to.match(/^-----BEGIN PGP MESSAGE/);
@ -339,7 +336,7 @@ describe('High level API', function() {
}); });
it('RSA: signClearMessage async', function (done) { it('RSA: signClearMessage async', function (done) {
proxy.signClearMessage([privKeyRSA], plaintext, function(err, data) { openpgp.signClearMessage([privKeyRSA], plaintext, function(err, data) {
expect(err).to.not.exist; expect(err).to.not.exist;
expect(data).to.exist; expect(data).to.exist;
expect(data).to.match(/-----BEGIN PGP SIGNED MESSAGE-----/); expect(data).to.match(/-----BEGIN PGP SIGNED MESSAGE-----/);
@ -350,7 +347,7 @@ describe('High level API', function() {
}); });
it('DSA: signClearMessage async', function (done) { it('DSA: signClearMessage async', function (done) {
proxy.signClearMessage([privKeyDE], plaintext, function(err, data) { openpgp.signClearMessage([privKeyDE], plaintext, function(err, data) {
expect(err).to.not.exist; expect(err).to.not.exist;
expect(data).to.exist; expect(data).to.exist;
expect(data).to.match(/-----BEGIN PGP SIGNED MESSAGE-----/); expect(data).to.match(/-----BEGIN PGP SIGNED MESSAGE-----/);
@ -363,7 +360,7 @@ describe('High level API', function() {
it('RSA: verifyClearSignedMessage async', function (done) { it('RSA: verifyClearSignedMessage async', function (done) {
var signed = openpgp.signClearMessage([privKeyRSA], plaintext); var signed = openpgp.signClearMessage([privKeyRSA], plaintext);
signed = openpgp.cleartext.readArmored(signed); signed = openpgp.cleartext.readArmored(signed);
proxy.verifyClearSignedMessage([pubKeyRSA], signed, function(err, data) { openpgp.verifyClearSignedMessage([pubKeyRSA], signed, function(err, data) {
expect(err).to.not.exist; expect(err).to.not.exist;
expect(data).to.exist; expect(data).to.exist;
verifySignature(data, privKeyRSA); verifySignature(data, privKeyRSA);
@ -378,7 +375,7 @@ describe('High level API', function() {
before(initKeys); before(initKeys);
it('Signing with not decrypted key gives error', function (done) { it('Signing with not decrypted key gives error', function (done) {
proxy.signClearMessage([privKeyRSA], plaintext, function(err, data) { openpgp.signClearMessage([privKeyRSA], plaintext, function(err, data) {
expect(data).to.not.exist; expect(data).to.not.exist;
expect(err).to.exist; expect(err).to.exist;
expect(err.message).to.equal('Private key is not decrypted.'); expect(err.message).to.equal('Private key is not decrypted.');
@ -404,7 +401,7 @@ describe('High level API', function() {
describe('Key generation', function() { describe('Key generation', function() {
it('Generate 1024-bit RSA/RSA key async', function (done) { it('Generate 1024-bit RSA/RSA key async', function (done) {
proxy.generateKeyPair(3, 1024, 'Test McTestington <test@example.com>', 'hello world', function(err, data) { openpgp.generateKeyPair(3, 1024, 'Test McTestington <test@example.com>', 'hello world', function(err, data) {
expect(err).to.not.exist; expect(err).to.not.exist;
expect(data).to.exist; expect(data).to.exist;
expect(data.publicKeyArmored).to.match(/^-----BEGIN PGP PUBLIC/); expect(data.publicKeyArmored).to.match(/^-----BEGIN PGP PUBLIC/);
@ -426,9 +423,10 @@ describe('High level API', function() {
describe('Decrypt secret key', function() { describe('Decrypt secret key', function() {
var msg; var msg, proxy;
beforeEach(function() { beforeEach(function() {
proxy = new openpgp.AsyncProxy('../dist/openpgp.worker.js');
initKeys(); initKeys();
msg = openpgp.message.fromText(plaintext).encrypt([pubKeyRSA]); msg = openpgp.message.fromText(plaintext).encrypt([pubKeyRSA]);
}); });