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

clnrest-rs #598

Merged
merged 1 commit into from
Oct 1, 2024
Merged

clnrest-rs #598

merged 1 commit into from
Oct 1, 2024

Conversation

daywalker90
Copy link
Contributor

ontop of #594 so i don't have to do a follow up

Changelog

[0.2.0] 2024-09-26

Added

  • log rune id, method and params when access was granted

Changed

  • clnrest-cors-origins is now a multi option, meaning you have to specify it multiple times if you have multiple values, you must no longer feed it multiple values in a single string. This change was missing to be a drop-in replacement with clnrest.py

@daywalker90 daywalker90 force-pushed the clnrest-rs branch 3 times, most recently from 89be10b to 09baba5 Compare September 26, 2024 18:55
@chrisguida
Copy link
Collaborator

Tested on my nixos mainnet box with Zeus, works great!

@daywalker90 I would add docs for the config in the README, especially clnrest-port, which the plugin cannot start without.

ACK 09baba5

@chrisguida
Copy link
Collaborator

@cdecker @m-schmoock @vincenzopalazzo merging this soon unless I hear otherwise! I think it's a fantastic plugin.

@chrisguida chrisguida merged commit 061abc1 into lightningd:master Oct 1, 2024
7 checks passed
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.

2 participants