keeweb
/
keeweb
mirror of https://github.com/keeweb/keeweb.git
1
0
Fork 0
Code Issues Releases Wiki Activity

fix #1252: public key rotation

This commit is contained in:
antelle 2019-09-28 14:40:46 +02:00
parent 2913a0421d
commit 0cd401c45d
6 changed files with 34 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
app/resources/public-key-new.pem Normal file
View File

@ -0,0 +1,9 @@
-----BEGIN PUBLIC KEY-----
MIIBIjANBgkqhkiG9w0BAQEFAAOCAQ8AMIIBCgKCAQEAnRq2k3TTx0ewTe6wDr6Q
VeB5diwiIWzsJD+ApfZu1KNPedcAgslAfjpNsYF1if6cYsPMJH70xJ2np6RQBl1V
PdwShOuxkD7m0BD5Hw/Aar8Hdp5cvAdOOMdBO+0DbGeUMy+z66s+oUCJmqVp19T6
PkkxbhN08rgtT7v+aFvrbqbO/vlsskbJpH2K2io+e1XmRGPnSr9q4KSqfGbTfe5g
LwDIOFd66Z4mb5Utb5wWpsy6Gjh06Yf257AccGD3A1bkTNOyeeX0tqciYBePWMk0
icP/aZ6hnErfhnUKf3tOgPLppSHiGcaSKekhChZ2xLUs3U64JwrXSmwHj+TzdO3S
0QIDAQAB
-----END PUBLIC KEY-----

8
app/scripts/plugins/plugin-manager.js
View File

@ -191,10 +191,10 @@ class PluginManager extends Model {
const galleryPlugin = gallery
? gallery.plugins.find(pl => pl.manifest.name === desc.manifest.name)
: null;
const expectedPublicKey = galleryPlugin
? galleryPlugin.manifest.publicKey
: SignatureVerifier.getPublicKey();
enabled = desc.manifest.publicKey === expectedPublicKey;
const expectedPublicKeys = galleryPlugin
? [galleryPlugin.manifest.publicKey]
: SignatureVerifier.getPublicKeys();
enabled = expectedPublicKeys.includes(desc.manifest.publicKey);
}
return plugin
.install(enabled, true)

2
app/scripts/plugins/plugin.js
View File

@ -106,7 +106,7 @@ class Plugin extends Model {
}
if (
!this.skipSignatureValidation &&
manifest.publicKey !== SignatureVerifier.getPublicKey()
!SignatureVerifier.getPublicKeys().includes(manifest.publicKey)
) {
return 'Public key mismatch';
}

29
app/scripts/util/data/signature-verifier.js
View File

@ -1,19 +1,26 @@
import kdbxweb from 'kdbxweb';
import { Logger } from 'util/logger';
import publicKey from 'public-key.pem';
import publicKeyData from 'public-key.pem';
import publicKeyDataNew from 'public-key-new.pem';
const SignatureVerifier = {
logger: new Logger('signature-verifier'),
publicKey: null,
publicKeys: null,
verify(data, signature, pk) {
if (!pk) {
const pks = this.getPublicKeys();
return this.verify(data, signature, pks[0]).then(isValid => {
if (isValid || !pks[1]) {
return isValid;
}
return this.verify(data, signature, pks[1]);
});
}
return new Promise((resolve, reject) => {
const algo = { name: 'RSASSA-PKCS1-v1_5', hash: { name: 'SHA-256' } };
try {
if (!pk) {
pk = this.getPublicKey();
}
if (typeof signature === 'string') {
signature = kdbxweb.ByteUtils.base64ToBytes(signature);
}
@ -54,13 +61,13 @@ const SignatureVerifier = {
});
},
getPublicKey() {
if (!this.publicKey) {
this.publicKey = publicKey
.match(/-+BEGIN PUBLIC KEY-+([\s\S]+?)-+END PUBLIC KEY-+/)[1]
.replace(/\s+/g, '');
getPublicKeys() {
if (!this.publicKeys) {
this.publicKeys = [publicKeyData, publicKeyDataNew].map(pk =>
pk.match(/-+BEGIN PUBLIC KEY-+([\s\S]+?)-+END PUBLIC KEY-+/)[1].replace(/\s+/g, '')
);
}
return this.publicKey;
return this.publicKeys;
}
};

1
build/webpack.config.js
View File

@ -70,6 +70,7 @@ function config(grunt, mode = 'production') {
'argon2-wasm': 'argon2-browser/dist/argon2.wasm',
templates: path.join(rootDir, 'app/templates'),
'public-key.pem': path.join(rootDir, 'app/resources/public-key.pem'),
'public-key-new.pem': path.join(rootDir, 'app/resources/public-key-new.pem'),
'demo.kdbx': path.join(rootDir, 'app/resources/Demo.kdbx')
}
},

1
release-notes.md
View File

@ -14,6 +14,7 @@ Release notes
`+` #498: CSV import
`*` #156: using ServiceWorker instead of AppCache
`+` #564: Steam OTP support
`+` #1252: public key rotation
`*` devtools are now opened with alt-cmd-I
`-` fix #764: multiple attachments display
`-` fix multi-line fields display in history