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

Need a --rpc-login option for monerod rpc #41

Closed
Tomahna81 opened this issue Sep 27, 2021 · 6 comments · Fixed by #169
Closed

Need a --rpc-login option for monerod rpc #41

Tomahna81 opened this issue Sep 27, 2021 · 6 comments · Fixed by #169

Comments

@Tomahna81
Copy link

It seems that the current implementation does not let the user run p2pool with a monerod rpc which has restricted access by password / login. Would be very useful (at least for me)...

@f-fusco
Copy link

f-fusco commented Jan 9, 2022

Would be nice to have. Anyone following the official guide for connecting to a node over Tor can't use p2pool, unless by omitting the rpc-login option.

@adhisimon
Copy link

I am also waiting for this feature

@bladedoyle
Copy link
Contributor

bladedoyle commented Jun 1, 2022

monero node rpc uses "digest" auth (as opposed to Basicauth)

@bladedoyle
Copy link
Contributor

bladedoyle commented Jun 3, 2022

@SChernykh I think I can take this task and produce a PR if this is a feature you want added?

edit: Ya, it looks like this is more work than im prepared to take on. I am thinking it should be possible to write a simple shim/proxy to sit between the p2pool and the monero daemon that handles the digest auth....but built-in support would be much, much better and its June already so I'll wait on your work.

edit: I think instead I'll put a little time into writing a "./configure" script for interactive customization of the docker-compose configs. Will submit PR for this in a week or so.

@SChernykh
Copy link
Owner

@bladedoyle This is not an easy task. Just read through https://datatracker.ietf.org/doc/html/rfc7616 to get an idea. JSONRPCRequest class does requests at a very low level and uses external/src/llhttp to parse monerod responses. Implementing RFC 7616 is a lot of work, and bringing the whole libcurl just to enable rpc login seems an overkill. Also, in case with libcurl, it can't do HTTP requests in a fully non-blocking way, so additional workarounds (like uv queue_work) will be needed.

@SChernykh
Copy link
Owner

It looks like curl can work with libuv properly: https://curl.se/libcurl/c/multi-uv.html
This is still a huge task. I'll look into it in June.

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 a pull request may close this issue.

5 participants