jstefanop / Apollo-Miner-Binaries

Binaries for running the standard unit Apollo BTC on a PC, Laptop, or Raspberry Pi
http://www.futurebit.io
26 stars 7 forks source link

stratumv1 implementation missing #13

Open sysmanalex opened 1 year ago

sysmanalex commented 1 year ago

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 commented 1 year ago

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

sysmanalex commented 1 year ago

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 commented 1 year ago

thanks! will submit a fix on the next release