index zap senders with P tag

[pablof7z]

A lot of blood has been shed by the lack of an index to find zaps sent by a pubkey.

Mutiny wallet is running a primal index server just to be able to run this simple query.

CHILDREN ARE DYING!

[TonyGiorgio]

We use primal for zaps sent and zaps received by collection of pubkeys.

Specifically interested in all economic activity by someone's follow list.

[benthecarman]

Definite concept ack

[jb55]

ack. huge oversight in the original spec.

[benthecarman]

the uppercase P can be pretty confusing, might be better as something like sender

[jb55]

On Sat, Dec 30, 2023 at 08:38:35PM -0800, benthecarman wrote: the uppercase `P` can be pretty confusing, might be better as something like `sender`

Then it wouldn't be efficiently queryable. most relays only index single-char tags.

[v0l]

Probably better if zap services can add this tag to the receipt and its not needed in zap request at all?

I already implemented this for snort / zap.stream zaps

[vitorpamplona]

LGTM

It is worth mentioning that the Zapper service can P-tag incorrectly and/or malicious actors can use this to spam everyone. In other words, the new tag is just a hint to who the sender is and not proof that that person sent it.

[v0l]

LGTM

It is worth mentioning that the Zapper service can P-tag incorrectly and/or malicious actors can use this to spam everyone. In other words, the new tag is just a hint to who the sender is and not proof that that person sent it.

zaps are not proof of anything

[benthecarman]

Implemented in rust-nostr: rust-nostr/nostr#230

[pablof7z]

Implemented in nostdress: believethehype/nostdress#28

P-tagged Zap: https://njump.me/nevent1qqs2rnyrwjf3yfl6h7j0wg2zke6p0unh34c8pupnmwcll4uqascfm8qprpmhxue69uhhyetvv9ujumn0wdmksetjv5hxxmmdqgsgyq7wmt5892vd437h99ta4sd8gp5x74pmy2wdn0r3vq639dn6qls4tujt9

[yukibtc]

I see the P tag really confusing.

What if we use the d tag, composed with a prefix + the pubkey? Something like: ["d", "zap-sender:<public-key>"]?

So the filter will be: { "kind": 9735, "#d": "zap-sender:<pk>" }

What do you think?

[benthecarman]

For added context, if you look at my rust-nostr PR, this change is really ugly for typed languages where we already have a type defined for p, so we need to name this one UpperP