diff --git a/src/key.js b/src/key.js index afbc1f89..00d2ba03 100644 --- a/src/key.js +++ b/src/key.js @@ -440,12 +440,42 @@ Key.prototype.verifyPrimaryKey = function() { } // check V4 expiration time if (this.primaryKey.version == 4 && primaryUser.selfCertificate.keyNeverExpires === false && - Date.now() > (primaryUser.selfCertificate.created.getTime() + primaryUser.selfCertificate.keyExpirationTime*1000)) { + Date.now() > (this.primaryKey.created.getTime() + primaryUser.selfCertificate.keyExpirationTime*1000)) { return enums.keyStatus.expired; } return enums.keyStatus.valid; }; +/** + * Returns the expiration time of the primary key or null if key does not expire + * @return {Date|null} + */ +Key.prototype.getExpirationTime = function() { + if (this.primaryKey.version == 3) { + return getExpirationTime(this.primaryKey); + } + if (this.primaryKey.version == 4) { + var primaryUser = this.getPrimaryUser(); + if (!primaryUser) { + return null; + } + return getExpirationTime(this.primaryKey, primaryUser.selfCertificate); + } +}; + + +function getExpirationTime(keyPacket, selfCertificate) { + // check V3 expiration time + if (keyPacket.version == 3 && keyPacket.expirationTimeV3 !== 0) { + return new Date(keyPacket.created.getTime() + keyPacket.expirationTimeV3*24*3600*1000); + } + // check V4 expiration time + if (keyPacket.version == 4 && selfCertificate.keyNeverExpires === false) { + return new Date(keyPacket.created.getTime() + selfCertificate.keyExpirationTime*1000); + } + return null; +}; + /** * Returns primary user and most significant (latest valid) self signature * - if multiple users are marked as primary users returns the one with the latest self signature @@ -668,6 +698,14 @@ SubKey.prototype.verify = function(primaryKey) { return enums.keyStatus.valid; }; +/** + * Returns the expiration time of the subkey or null if key does not expire + * @return {Date|null} + */ +SubKey.prototype.getExpirationTime = function() { + return getExpirationTime(this.subKey, this.bindingSignature); +}; + /** * Reads an OpenPGP armored text and returns one or multiple key objects * @param {String} armoredText text to be parsed diff --git a/test/general/key.js b/test/general/key.js index c3b52c07..151b847a 100644 --- a/test/general/key.js +++ b/test/general/key.js @@ -308,5 +308,19 @@ describe('Key', function() { expect(keyPacket).to.not.exist; }); + it('Method getExpirationTime V4 Key', function() { + var pubKey = openpgp.key.readArmored(twoKeys).keys[1]; + expect(pubKey).to.exist; + expect(pubKey).to.be.an.instanceof(openpgp.key.Key); + expect(pubKey.getExpirationTime().toISOString()).to.be.equal('2018-11-26T10:58:29.000Z'); + }); + + it('Method getExpirationTime V4 SubKey', function() { + var pubKey = openpgp.key.readArmored(twoKeys).keys[1]; + expect(pubKey).to.exist; + expect(pubKey).to.be.an.instanceof(openpgp.key.Key); + expect(pubKey.subKeys[0].getExpirationTime().toISOString()).to.be.equal('2018-11-26T10:58:29.000Z'); + }); + });