search

nostr-dev-kit / ndk

Nostr Development Kit with outbox-model support
https://nostr-dev-kit.github.io/ndk/
MIT License
366 stars 99 forks source link

Fix eventsPerRelay property to include dup events #218

Closed melari closed 6 months ago

melari commented 6 months ago

After https://github.com/nostr-dev-kit/ndk/commit/7e50e4bf7891854cf9c9d5eef6b53e66efc81ca3 the early return for dups skips updating the eventsPerRelay property on the subscription. I believe this was unintended since the commit description mentions only skipping the signature verification.

This PR restores the eventsPerRelay feature by tracking duplicates as well.

pablof7z commented 6 months ago

thx for catching that; I was wondering why that stopped working but hadn't had the time to look into it