Found a bug in the nostr marketplace (nip?) today. Seems like strfry can't parse these events. It throws 'std::get wrong index for variant' So that may mean the marketplace is gonna be hard to get your events out when so much of nostr runs strfry. π€ Is this why my moon socks aren't selling? ππ
Discussion
i hate variants
they are a violation of static typing and the worst mistake of C was union types (the C form of the variant)
My absolutely shot in the dark guess is that maybe strfry doesn't like tags that don't have single char (like, e or p or ..) as the first element in the tag array.. just a guess. Marketplace seems to have whole words as the first element. I don't see much else that would be tripping it up..
Will prob do more testing later to see if this theory is correct .. been seeing this error a while, finally I'm on the case with an event that can cause it π
the spec only says that only single letter tags have to be *indexed* not that any such tags other than a-zA-Z are invalid
it isn't supposed to index these tags, and if the spec for marketplaces depends on non-single-alphabetical-letter tags then it is wrong
like this one:
https://github.com/nostr-protocol/nips/blob/d251ca0da5372ba0ae0df76754d96068266a6778/24.md
this "title" tag key is wrong, for example, i don't know why it isn't `T` that is the logical one to use for that
just to clarify, what the spec states about indexing means that you can't search for this tag using a filter
Is strfry still being developed? I thought the author moved on to other stuff.
Last commit was 6mo ago, so not quite unmaintained but... Ya I dunno. Sometimes apps don't need changes for long periods π
