Keep signed prekeys for one month

Co-authored-by: Scott Nonnenberg <scott@signal.org>
This commit is contained in:
Ehren Kret 2020-12-17 17:29:20 -06:00 committed by Scott Nonnenberg
parent 8144d758ae
commit 38ab92da5d
2 changed files with 18 additions and 18 deletions

View file

@ -48,22 +48,22 @@ describe('AccountManager', () => {
}); });
}); });
it('keeps three confirmed keys even if over a week old', () => { it('keeps three confirmed keys even if over a month old', () => {
const now = Date.now(); const now = Date.now();
signedPreKeys = [ signedPreKeys = [
{ {
keyId: 1, keyId: 1,
created_at: now - DAY * 21, created_at: now - DAY * 32,
confirmed: true, confirmed: true,
}, },
{ {
keyId: 2, keyId: 2,
created_at: now - DAY * 14, created_at: now - DAY * 34,
confirmed: true, confirmed: true,
}, },
{ {
keyId: 3, keyId: 3,
created_at: now - DAY * 18, created_at: now - DAY * 38,
confirmed: true, confirmed: true,
}, },
]; ];
@ -72,27 +72,27 @@ describe('AccountManager', () => {
return accountManager.cleanSignedPreKeys(); return accountManager.cleanSignedPreKeys();
}); });
it('eliminates confirmed keys over a week old, if more than three', async () => { it('eliminates confirmed keys over a month old, if more than three', async () => {
const now = Date.now(); const now = Date.now();
signedPreKeys = [ signedPreKeys = [
{ {
keyId: 1, keyId: 1,
created_at: now - DAY * 21, created_at: now - DAY * 32,
confirmed: true, confirmed: true,
}, },
{ {
keyId: 2, keyId: 2,
created_at: now - DAY * 14, created_at: now - DAY * 31,
confirmed: true, confirmed: true,
}, },
{ {
keyId: 3, keyId: 3,
created_at: now - DAY * 4, created_at: now - DAY * 24,
confirmed: true, confirmed: true,
}, },
{ {
keyId: 4, keyId: 4,
created_at: now - DAY * 18, created_at: now - DAY * 38,
confirmed: true, confirmed: true,
}, },
{ {
@ -120,19 +120,19 @@ describe('AccountManager', () => {
signedPreKeys = [ signedPreKeys = [
{ {
keyId: 1, keyId: 1,
created_at: now - DAY * 14, created_at: now - DAY * 32,
}, },
{ {
keyId: 2, keyId: 2,
created_at: now - DAY * 21, created_at: now - DAY * 44,
}, },
{ {
keyId: 3, keyId: 3,
created_at: now - DAY * 18, created_at: now - DAY * 36,
}, },
{ {
keyId: 4, keyId: 4,
created_at: now - DAY, created_at: now - DAY * 20,
}, },
]; ];
@ -154,21 +154,21 @@ describe('AccountManager', () => {
signedPreKeys = [ signedPreKeys = [
{ {
keyId: 1, keyId: 1,
created_at: now - DAY * 21, created_at: now - DAY * 32,
confirmed: true, confirmed: true,
}, },
{ {
keyId: 2, keyId: 2,
created_at: now - DAY * 14, created_at: now - DAY * 44,
confirmed: true, confirmed: true,
}, },
{ {
keyId: 3, keyId: 3,
created_at: now - DAY * 12, created_at: now - DAY * 36,
}, },
{ {
keyId: 4, keyId: 4,
created_at: now - DAY * 8, created_at: now - DAY * 20,
}, },
]; ];

View file

@ -18,7 +18,7 @@ import WebSocketResource, {
IncomingWebSocketRequest, IncomingWebSocketRequest,
} from './WebsocketResources'; } from './WebsocketResources';
const ARCHIVE_AGE = 7 * 24 * 60 * 60 * 1000; const ARCHIVE_AGE = 30 * 24 * 60 * 60 * 1000;
function getIdentifier(id: string) { function getIdentifier(id: string) {
if (!id || !id.length) { if (!id || !id.length) {