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

stratumv1 implementation missing #13

Open
sysmanalex opened this issue Apr 20, 2023 · 3 comments
Open

stratumv1 implementation missing #13

sysmanalex opened this issue Apr 20, 2023 · 3 comments

Comments

@sysmanalex
Copy link

WARNING: Unknown notify (mining.ping):
(3){params=(0)[],id=42,method="mining.ping"}

is there possibility to obtain source and fix some issues there ?

@jstefanop
Copy link
Owner

Hey we can take a look, can you describe the issue or provide a log?

@sysmanalex
Copy link
Author

all simple
Miner: >> {"result":true,"error":null,"id":31337}
Server: << {"params":[],"id":42,"method":"mining.ping"}

Apollo miner, reporting WARNING: Unknown notify (mining.ping):
normally miners responding pong, like cgminer - answering

{"id": 42, "result": "pong", "error": null}"

check cgminer by ckolivas here https://github.com/ckolivas/cgminer/blob/master/util.c#L2708
send_pong func
https://github.com/ckolivas/cgminer/blob/master/util.c#L2589

not a critical one, but avoid warnings + some pools measure clients reponse time + without 15 no responses, may close connection.

@jstefanop
Copy link
Owner

thanks! will submit a fix on the next release

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

No branches or pull requests

2 participants