adamritter / nostr-relaypool-ts

A Nostr RelayPool implementation in TypeScript using only nostr-tools library as a dependency
https://www.npmjs.com/package/nostr-relaypool
MIT License
63 stars 17 forks source link

Think about removing relay property from filter after delay is implemented #12

Open adamritter opened 1 year ago

adamritter commented 1 year ago

After delay is implemented, filtering just for a relay won't be costly, thereby the relay property isn't really needed, and just makes the API more complicated. It can probably be removed, but it's good to evaluate on a reference implementation. Anyways it's probably better to leave it, as different implementations may use different methods for subscribing.