feat!: remove nip-26 delegation support (#350)

BREAKING CHANGE: NIP-26 support has been removed. Delegated events will not be handled differently.

Author: cameri

README.md

@@ -57,7 +57,7 @@ NIPs with a relay-specific implementation are listed here.
 - [x] NIP-16: Event Treatment
 - [x] NIP-20: Command Results
 - [x] NIP-22: Event `created_at` Limits
-- [x] NIP-26: Delegated Event Signing
+- [ ] NIP-26: Delegated Event Signing (REMOVED)
 - [x] NIP-28: Public Chat
 - [x] NIP-33: Parameterized Replaceable Events
 - [x] NIP-40: Expiration Timestamp

migrations/20240111204900_remove_delegator_from_events_table.js

@@ -0,0 +1,18 @@
+exports.up = async function (knex) {
+  await knex.schema
+    .raw('DROP INDEX IF EXISTS pubkey_delegator_kind_idx;')
+  await knex.schema.alterTable('events', function (table) {
+    table.dropColumn('event_delegator')
+  })
+}
+
+exports.down = async function (knex) {
+  await knex.schema.alterTable('events', function (table) {
+    table.binary('event_delegator').nullable().index()
+  })
+  await knex.schema
+    .raw(
+      `CREATE UNIQUE INDEX pubkey_delegator_kind_idx
+      ON events ( event_pubkey, event_delegator, event_kind );`,
+    )
+}

package.json

@@ -13,7 +13,6 @@
     16,
     20,
     22,
-    26,
     28,
     33,
     40

src/@types/event.ts

@@ -1,5 +1,5 @@
 import { ContextMetadata, EventId, Pubkey, Tag } from './base'
-import { ContextMetadataKey, EventDeduplicationMetadataKey, EventDelegatorMetadataKey, EventExpirationTimeMetadataKey, EventKinds } from '../constants/base'
+import { ContextMetadataKey, EventDeduplicationMetadataKey, EventExpirationTimeMetadataKey, EventKinds } from '../constants/base'
 
 export interface BaseEvent {
   id: EventId
@@ -21,10 +21,6 @@ export type UnsignedEvent = Omit<Event, 'sig'>
 
 export type UnidentifiedEvent = Omit<UnsignedEvent, 'id'>
 
-export interface DelegatedEvent extends Event {
-  [EventDelegatorMetadataKey]?: Pubkey
-}
-
 export interface ExpiringEvent extends Event {
   [EventExpirationTimeMetadataKey]?: number
 }
@@ -42,7 +38,6 @@ export interface DBEvent {
   event_content: string
   event_tags: Tag[]
   event_signature: Buffer
-  event_delegator?: Buffer | null
   event_deduplication?: string | null
   first_seen: Date
   deleted_at?: Date

src/@types/subscription.ts

@@ -5,7 +5,7 @@ export type SubscriptionId = string
 
 export interface SubscriptionFilter {
   ids?: EventId[]
-  kinds?: EventKinds[]
+  kinds?: (EventKinds | number)[]
   since?: number
   until?: number
   authors?: Pubkey[]

src/constants/base.ts

@@ -37,7 +37,6 @@ export enum EventTags {
   Event = 'e',
   Pubkey = 'p',
   //  Multicast = 'm',
-  Delegation = 'delegation',
   Deduplication = 'd',
   Expiration = 'expiration',
   Invoice = 'bolt11',
@@ -49,7 +48,6 @@ export enum PaymentsProcessors {
   LNBITS = 'lnbits',
 }
 
-export const EventDelegatorMetadataKey = Symbol('Delegator')
 export const EventDeduplicationMetadataKey = Symbol('Deduplication')
 export const ContextMetadataKey = Symbol('Context')
 export const EventExpirationTimeMetadataKey = Symbol('Expiration')

src/factories/delegated-event-strategy-factory.ts

@@ -1,11 +1,8 @@
 import { IEventRepository, IUserRepository } from '../@types/repositories'
 import { IncomingMessage, MessageType } from '../@types/messages'
 import { createSettings } from './settings-factory'
-import { DelegatedEventMessageHandler } from '../handlers/delegated-event-message-handler'
-import { delegatedEventStrategyFactory } from './delegated-event-strategy-factory'
 import { EventMessageHandler } from '../handlers/event-message-handler'
 import { eventStrategyFactory } from './event-strategy-factory'
-import { isDelegatedEvent } from '../utils/event'
 import { IWebSocketAdapter } from '../@types/adapters'
 import { slidingWindowRateLimiterFactory } from './rate-limiter-factory'
 import { SubscribeMessageHandler } from '../handlers/subscribe-message-handler'
@@ -18,16 +15,6 @@ export const messageHandlerFactory = (
   switch (message[0]) {
     case MessageType.EVENT:
       {
-        if (isDelegatedEvent(message[1])) {
-          return new DelegatedEventMessageHandler(
-            adapter,
-            delegatedEventStrategyFactory(eventRepository),
-            userRepository,
-            createSettings,
-            slidingWindowRateLimiterFactory,
-          )
-        }
-
         return new EventMessageHandler(
           adapter,
           eventStrategyFactory(eventRepository),
@@ -39,7 +26,7 @@ export const messageHandlerFactory = (
     case MessageType.REQ:
       return new SubscribeMessageHandler(adapter, eventRepository, createSettings)
     case MessageType.CLOSE:
-      return new UnsubscribeMessageHandler(adapter,)
+      return new UnsubscribeMessageHandler(adapter)
     default:
       throw new Error(`Unknown message type: ${String(message[0]).substring(0, 64)}`)
   }

src/factories/message-handler-factory.ts

@@ -28,7 +28,7 @@ import {
   toPairs,
 } from 'ramda'
 
-import { ContextMetadataKey, EventDeduplicationMetadataKey, EventDelegatorMetadataKey, EventExpirationTimeMetadataKey } from '../constants/base'
+import { ContextMetadataKey, EventDeduplicationMetadataKey, EventExpirationTimeMetadataKey } from '../constants/base'
 import { DatabaseClient, EventId } from '../@types/base'
 import { DBEvent, Event } from '../@types/event'
 import { IEventRepository, IQueryResult } from '../@types/repositories'
@@ -105,7 +105,7 @@ export class EventRepository implements IEventRepository {
           ])(currentFilter[filterName] as string[])
         })
       })({
-        authors: ['event_pubkey', 'event_delegator'],
+        authors: ['event_pubkey'],
         ids: ['event_id'],
       })
 
@@ -180,11 +180,6 @@ export class EventRepository implements IEventRepository {
       event_tags: pipe(prop('tags'), toJSON),
       event_content: prop('content'),
       event_signature: pipe(prop('sig'), toBuffer),
-      event_delegator: ifElse(
-        propSatisfies(is(String), EventDelegatorMetadataKey),
-        pipe(prop(EventDelegatorMetadataKey as any), toBuffer),
-        always(null),
-      ),
       remote_address: path([ContextMetadataKey as any, 'remoteAddress', 'address']),
       expires_at: ifElse(
         propSatisfies(is(Number), EventExpirationTimeMetadataKey),
@@ -212,11 +207,6 @@ export class EventRepository implements IEventRepository {
       event_tags: pipe(prop('tags'), toJSON),
       event_content: prop('content'),
       event_signature: pipe(prop('sig'), toBuffer),
-      event_delegator: ifElse(
-        propSatisfies(is(String), EventDelegatorMetadataKey),
-        pipe(prop(EventDelegatorMetadataKey as any), toBuffer),
-        always(null),
-      ),
       event_deduplication: ifElse(
         propSatisfies(isNil, EventDeduplicationMetadataKey),
         pipe(paths([['pubkey'], ['kind']]), toJSON),