Merge branch 'created-at-up' into 'main'

pipeline: move created_at check up

See merge request soapbox-pub/ditto!336
This commit is contained in:
Alex Gleason 2024-05-30 17:37:38 +00:00
commit e543e4c2a5
1 changed files with 4 additions and 5 deletions

View File

@ -28,6 +28,10 @@ const debug = Debug('ditto:pipeline');
* It is idempotent, so it can be called multiple times for the same event.
*/
async function handleEvent(event: DittoEvent, signal: AbortSignal): Promise<void> {
// Integer max value for Postgres. TODO: switch to a bigint in 2038.
if (event.created_at >= 2_147_483_647) {
throw new RelayError('blocked', 'event too far in the future');
}
if (!(await verifyEventWorker(event))) return;
if (encounterEvent(event)) return;
debug(`NostrEvent<${event.kind}> ${event.id}`);
@ -115,11 +119,6 @@ async function storeEvent(event: DittoEvent, signal?: AbortSignal): Promise<void
const store = await Storages.db();
const kysely = await DittoDB.getInstance();
// Integer max value for Postgres. TODO: switch to a bigint in 2038.
if (event.created_at >= 2_147_483_647) {
throw new RelayError('blocked', 'event too far in the future');
}
await updateStats({ event, store, kysely }).catch(debug);
await store.event(event, { signal });
}