suzanne.soy/fork-openpgpjs
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Redefine Regex for blank line after armor header. This fixes an issue with truncated blank lines at the beginning of cleartext signed messages.

Browse Source
This commit is contained in:
Thomas Oberndörfer 2014-05-16 19:08:29 +02:00
parent f41e644259
commit 172855a8e9
2 changed files with 9 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/encoding/armor.js
View File

@ -211,7 +211,8 @@ function createcrc24(input) {
* and an attribute "body" containing the body.
*/
function splitHeaders(text) {
var reEmptyLine = /^\s*\n/m;
// empty line with whitespace characters
var reEmptyLine = /^[ \f\r\t\u00a0\u2000-\u200a\u202f\u205f\u3000]*\n/m;
var headers = '';
var body = text;

8
test/general/armor.js
View File

@ -84,7 +84,7 @@ describe("ASCII armor", function() {
var msg =
['-----BEGIN PGP SIGNED MESSAGE-----',
'Hash: SHA1',
'\u000b\u00a0',
' \f\r\t\u00a0\u2000\u2001\u2002\u2003\u2004\u2005\u2006\u2007\u2008\u2009\u200a\u202f\u205f\u3000',
'sign this',
'-----BEGIN PGP SIGNATURE-----',
'Version: GnuPG v2.0.22 (GNU/Linux)',
@ -183,6 +183,12 @@ describe("ASCII armor", function() {
expect(result.keys[0]).to.be.an.instanceof(openpgp.key.Key);
});
it('Do not filter blank lines after header', function () {
var msg = getArmor(['Hash: SHA1', '']);
msg = openpgp.cleartext.readArmored(msg);
expect(msg.text).to.equal('\r\nsign this');
});
});