nostr:nevent1 render sometimes broken

[alltheseas]

https://damus.io/note1dyn6qnlqm3rwnk9quajjdn2eh8ldlflt6f8szhmkjdnplyy2w0kqm583w3

[kernelkind]

The nevent mention is valid, it's rendered correctly on Coracle

[jb55]

looks like it has 16 relays, I wonder if it's hitting our limit (although I thought it was 16). Maybe we can just bump it to 32

[jb55]

also we shouldn't fail to parse if we hit the limit, maybe that is happening as well.

[kernelkind]

looks like it has 16 relays, I wonder if it's hitting our limit (although I thought it was 16). Maybe we can just bump it to 32

The TLV limit is 16, and it looks like the problem is that the TLV with the special type is the one that didn't fit in the buffer so the code thinks the mention is invalid. It looks like you increased the max TLV limit in the nostrdb changes to 32 so I think this specific issue would be fixed when the nostrdb changes are integrated into damus.

This would only be a bandaid fix though, to fix the root of this issue we would need to ensure if a special type TLV exists in the URI, it makes it to the TLV buffer

[alltheseas]

The nevent linked still does not render.

a quick test reveals nostr:nevent render works