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.11rc3 #99

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

Conversation

ksedgwic
Copy link
Collaborator

DO NOT MERGE THIS

ShahanaFarooqui and others added 16 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 <pet.v.ne@gmail.com>

Changelog-Fixes: Plugin: clnrest crashed on startup when gevent was
missing.
Signed-off-by: Peter Neuroth <pet.v.ne@gmail.com>
@ksedgwic ksedgwic force-pushed the 2023-11-remote-hsmd-v23.11rc3 branch 4 times, most recently from 3b2ba22 to 96afd92 Compare November 28, 2023 01:13
@ksedgwic ksedgwic force-pushed the 2023-11-remote-hsmd-v23.11rc3 branch from 96afd92 to fb47a15 Compare November 28, 2023 04:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
5 participants