Skip to content

Conversation

@ksedgwic
Copy link

DO NOT MERGE THIS

ShahanaFarooqui and others added 23 commits November 21, 2023 08:23
…fications_received_via_websocket` function for other rpc requests too
Reported-by: Shahana Farooqui
Changelog-Fixed: JSON-RPC: Plugin notification `msat` fields in `invoice_payment` and `invoice_created` hooks now a number, not a string with "msat" suffix.
Changelog-Fixed: JSON-RPC: Plugin hook `payment` `msat` field is now a number, not a string with "msat" suffix.
This fixes a crash on startup of core-lightning where gevent could not
be imported. This happens before sys is imported and throws us into the
except clause which calls sys.
By importing it explicitly in the except clause we are not dependend of
the order of imports in the try bracket.

Signed-off-by: Peter Neuroth <[email protected]>

Changelog-Fixes: Plugin: clnrest crashed on startup when gevent was
missing.
This caused a segmentation fault when calling `getcodexsecret` without
id.

Signed-off-by: Peter Neuroth <[email protected]>

Changelog-Fixed: Hsmtool: Fix segmentation fault when calling `getcodexsecret` without id.
I guess this was to honor Aditya, but it was also a wrong example.

Signed-off-by: Peter Neuroth <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants