-
Notifications
You must be signed in to change notification settings - Fork 107
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
JoinMarket: Add support for jmwalletd #699
base: master
Are you sure you want to change the base?
Conversation
27ff5b4
to
9cea6a5
Compare
47b5009
to
9457a4d
Compare
eb6763e
to
de77a06
Compare
Hello @erikarvstedt @jonasnick , This service should probably only be accessible locally. I don't see any reason to expose it via an onion address. Additionally, please take a look at the address option, which I have marked as Also, do you think it makes sense to add some documentation about this service to the README? Could you please review this and assist with the service hardening? Thank you! |
7135884
to
b6454fd
Compare
62a3378
to
4d34ced
Compare
Just wanted to checking in. What are the next steps or further organization to get this merged? |
Thanks @seberm for putting this together! |
4d34ced
to
df65f6b
Compare
df65f6b
to
e6e3a96
Compare
This PR adds support for jmwalletd daemon which is needed for joinmarket-jam to run.
Refs.: