search

hoytech / strfry

a nostr relay
GNU General Public License v3.0
489 stars 96 forks source link

Cron deletes events that are not expired #73

Closed Dreyhh closed 1 month ago

Dreyhh commented 1 year ago

It seems there is an issue with the deletion of expired events, namely, the Cronjob deletes events that have not expired yet. To reproduce: create an event with expiration tag of say 7 days from the event's creation date then insert any other event to the relay. The event with the expiration tag will be considered expired by the Cronjob (and deleted)

hoytech commented 1 year ago

I'm going to look into this ASAP -- it is probably a bug introduced in a recent refactor.

KoalaSat commented 1 month ago

Hello @hoytech , any updates on this? I'm facing this issue, I can help with code if necessary :)

KoalaSat commented 1 month ago

for the record, I found the bug. Tested it and it seems to work https://github.com/hoytech/strfry/pull/117

hoytech commented 1 month ago

Thanks, looking at it now for next release! :+1: