do not echo data back to caller for detached signing
This commit is contained in:
parent
2ffa221b21
commit
81a59b76b1
|
@ -314,13 +314,13 @@ export function sign({ data, privateKeys, armor=true, detached=false}) {
|
||||||
}
|
}
|
||||||
} else {
|
} else {
|
||||||
message = message.sign(privateKeys);
|
message = message.sign(privateKeys);
|
||||||
}
|
|
||||||
|
|
||||||
if (armor) {
|
if (armor) {
|
||||||
result.data = message.armor();
|
result.data = message.armor();
|
||||||
} else {
|
} else {
|
||||||
result.message = message;
|
result.message = message;
|
||||||
}
|
}
|
||||||
|
}
|
||||||
|
|
||||||
return result;
|
return result;
|
||||||
|
|
||||||
}, 'Error signing cleartext message');
|
}, 'Error signing cleartext message');
|
||||||
|
|
|
@ -969,7 +969,7 @@ describe('OpenPGP.js public api tests', function() {
|
||||||
publicKeys: publicKey.keys
|
publicKeys: publicKey.keys
|
||||||
};
|
};
|
||||||
return openpgp.sign(signOpt).then(function(signed) {
|
return openpgp.sign(signOpt).then(function(signed) {
|
||||||
verifyOpt.message = openpgp.cleartext.readArmored(signed.data);
|
verifyOpt.message = new openpgp.cleartext.CleartextMessage(plaintext);
|
||||||
verifyOpt.signature = openpgp.signature.readArmored(signed.signature);
|
verifyOpt.signature = openpgp.signature.readArmored(signed.signature);
|
||||||
return openpgp.verify(verifyOpt);
|
return openpgp.verify(verifyOpt);
|
||||||
}).then(function(verified) {
|
}).then(function(verified) {
|
||||||
|
@ -1009,7 +1009,7 @@ describe('OpenPGP.js public api tests', function() {
|
||||||
publicKeys: openpgp.key.readArmored(wrong_pubkey).keys
|
publicKeys: openpgp.key.readArmored(wrong_pubkey).keys
|
||||||
};
|
};
|
||||||
return openpgp.sign(signOpt).then(function(signed) {
|
return openpgp.sign(signOpt).then(function(signed) {
|
||||||
verifyOpt.message = openpgp.cleartext.readArmored(signed.data);
|
verifyOpt.message = new openpgp.cleartext.CleartextMessage(plaintext);
|
||||||
verifyOpt.signature = openpgp.signature.readArmored(signed.signature);
|
verifyOpt.signature = openpgp.signature.readArmored(signed.signature);
|
||||||
return openpgp.verify(verifyOpt);
|
return openpgp.verify(verifyOpt);
|
||||||
}).then(function(verified) {
|
}).then(function(verified) {
|
||||||
|
@ -1051,7 +1051,7 @@ describe('OpenPGP.js public api tests', function() {
|
||||||
publicKeys: publicKey.keys
|
publicKeys: publicKey.keys
|
||||||
};
|
};
|
||||||
return openpgp.sign(signOpt).then(function(signed) {
|
return openpgp.sign(signOpt).then(function(signed) {
|
||||||
verifyOpt.message = signed.message;
|
verifyOpt.message = new openpgp.cleartext.CleartextMessage(plaintext);
|
||||||
verifyOpt.signature = signed.signature;
|
verifyOpt.signature = signed.signature;
|
||||||
return openpgp.verify(verifyOpt);
|
return openpgp.verify(verifyOpt);
|
||||||
}).then(function(verified) {
|
}).then(function(verified) {
|
||||||
|
|
Loading…
Reference in New Issue
Block a user