Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

remote-hsmd on CLN v23.11 #100

Draft
wants to merge 23 commits into
base: master
Choose a base branch
from
Draft

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.

5 participants