Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

nip09: filtered deletation. #1509

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

kehiy
Copy link
Contributor

@kehiy kehiy commented Sep 21, 2024

using the current model, it's hard to remove events with complex conditions. client needs to turn user action into a huge event and send it to relay and then relay needs to do an iteration and remove them.

using f tag, we can make it much easier. filter already exists on relay and clients and they can parse, construct, and interpret them. for relys it's easy to turn this filter to a remove/delete query on different query languages compared with the current model.

then clients can support delete account, removing all chat messages, or even getting the filter as input from the user.

of course, the "authors" filed MUST be considered only if the matching events are kind of in the control of the "pubkey" field owner in the case of moderated communities and similar stuff.

@kehiy kehiy changed the title nip09: filter deletation. nip09: filtered deletation. Sep 21, 2024
09.md Show resolved Hide resolved
@vitorpamplona
Copy link
Collaborator

The only issue with broad deletions is that other relays will also delete this if the event gets rebroadcasted there. This is why on #1256 we had to specify the relay. Otherwise a database sync between relays might delete the filter elsewhere as well.

@kehiy
Copy link
Contributor Author

kehiy commented Sep 23, 2024

as i can see this nip-09 is intended to be used to delete something network-wide. if we want to perform such a thing, why not just make the deletation request a protected event then?

@kehiy
Copy link
Contributor Author

kehiy commented Sep 23, 2024

pr updated with better details and no typos.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants