brugeman on Nostr: Case in point - nip46 nip04_decrypt method used to return [plaintext], which didn't ...
Case in point - nip46 nip04_decrypt method used to return [plaintext], which didn't make sense, but that was spec-ed. Snort implemented a sane, but broken version - expecting just plaintext string. It wasn't working with ndk that returned [plaintext], so I looked things up, people were proposing to "fix" it - change the spec, I saw 'No it will break implementations' comment and went on to fix Snort - make it accept [plaintext]. Turns out that was "fixed" in the spec now and changed in NDK - it returns plaintext string now, but Snort has no idea about it and now I have to unfix Snort. I'm not sure, maybe I and Kieran aren't following the right process to stay up to date w/ NIP changes but this kinda illustrates the point.
Published at
2024-02-15 10:36:24Event JSON
{
"id": "b8ebeea0b5b862f1d58d738e053837ce9f54149c709cf4ec22dfb55824b593d1",
"pubkey": "3356de61b39647931ce8b2140b2bab837e0810c0ef515bbe92de0248040b8bdd",
"created_at": 1707993384,
"kind": 1,
"tags": [
[
"e",
"fc2820771c5dd59e0f789ef619534ddd17929feec1f9af33e8c2fc9a2945d316",
"wss://pyramid.fiatjaf.com/",
"root"
],
[
"e",
"cee237c52bfc1b6a53b2203864c8e70a964743a5f96e334487834da1aacd402d",
"wss://nos.lol/",
"reply"
],
[
"p",
"3bf0c63fcb93463407af97a5e5ee64fa883d107ef9e558472c4eb9aaaefa459d",
"",
"mention"
],
[
"p",
"ee11a5dff40c19a555f41fe42b48f00e618c91225622ae37b6c2bb67b76c4e49",
"",
"mention"
]
],
"content": "Case in point - nip46 nip04_decrypt method used to return [plaintext], which didn't make sense, but that was spec-ed. Snort implemented a sane, but broken version - expecting just plaintext string. It wasn't working with ndk that returned [plaintext], so I looked things up, people were proposing to \"fix\" it - change the spec, I saw 'No it will break implementations' comment and went on to fix Snort - make it accept [plaintext]. Turns out that was \"fixed\" in the spec now and changed in NDK - it returns plaintext string now, but Snort has no idea about it and now I have to unfix Snort. I'm not sure, maybe I and Kieran aren't following the right process to stay up to date w/ NIP changes but this kinda illustrates the point.",
"sig": "34a989d5ae1fbd3d24185a74a041abb99fed39fcfe8c119df4f97cdaf1e2a3b92178605222ac29588b3d1ee0c37edc2419a6dce343e9c3fdee3e7d20f8681956"
}