Mazin on Nostr: We took a long look at this today. There are multiple layers of our tech stack that ...
We took a long look at this today. There are multiple layers of our tech stack that do not support reason phrases anymore. There are some ways for us to hack it back in, but given that reason phrases are optional and not required by any other client or the spec, this would be a considerable amount of work with limited gain.
It seems like this php library doesn’t do anything with the reason phrase besides check that it exists so perhaps it should be made optional there?
Published at
2025-02-06 17:39:30Event JSON
{
"id": "7fde4aa818d05a62d65fb4ef8fb98c45cf7d9473bd72be3648cbe574f2152279",
"pubkey": "3d842afecd5e293f28b6627933704a3fb8ce153aa91d790ab11f6a752d44a42d",
"created_at": 1738863570,
"kind": 1,
"tags": [
[
"e",
"f3b025f1bc97ae7bfee91695e9f8a2b7789d110375323ea85b832ea3708524d5",
"",
"root"
],
[
"p",
"0dc2dcb14d89f94b8a1590e178e9fbcb2ef1cb0be175a283842f9dc54787801a"
],
[
"p",
"06639a386c9c1014217622ccbcf40908c4f1a0c33e23f8d6d68f4abf655f8f71"
]
],
"content": "We took a long look at this today. There are multiple layers of our tech stack that do not support reason phrases anymore. There are some ways for us to hack it back in, but given that reason phrases are optional and not required by any other client or the spec, this would be a considerable amount of work with limited gain. \n\nIt seems like this php library doesn’t do anything with the reason phrase besides check that it exists so perhaps it should be made optional there?",
"sig": "6f1742e9490c4898785d27985034e98a22a349578849693b1f9bef7a3761ce0bdd38a35c69f4c9a56922a5a5ac859a3bd0e38529b46c2ff59730cd55361679e6"
}
