bonesoul / CoiniumServ

Next-gen crypto currency mining pool software
Other
793 stars 494 forks source link

Vote - Remove getwork support #608

Open bonesoul opened 9 years ago

bonesoul commented 9 years ago

We do an initial implementation of getwork support (https://github.com/CoiniumServ/CoiniumServ/blob/2d4c8624dbfe156a0400683a4afed55a1a7154a1/src/CoiniumServ/Server/Mining/Getwork/GetworkService.cs) that's barely functional (actually mining &finding blocks is working but it can not communicate with rest of the systems).

I'm not aware of anyone that uses getwork support which is a quite old protocol. Today defacto standart is stratum. I'd like to also remove the support for getwork if the community agrees.

--- Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/5402353-vote-remove-getwork-support?utm_campaign=plugin&utm_content=tracker%2F401667&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F401667&utm_medium=issues&utm_source=github).
bonesoul commented 9 years ago

my own vote: +1

Caberhagen commented 9 years ago

+1 remove Am 20.10.2014 15:25 schrieb "Hüseyin Uslu" notifications@github.com:

my own vote: +1

— Reply to this email directly or view it on GitHub https://github.com/CoiniumServ/CoiniumServ/issues/608#issuecomment-59752957 .

djkyno commented 9 years ago

+1

Marty19 commented 9 years ago

+1

artbatista commented 9 years ago

Vote +1

Remove

frndxyz commented 9 years ago

+1

ahmedbodi commented 9 years ago

-1 we need it if we're having merged mining

bonesoul commented 9 years ago

Anyone using the merged-mining yet? is merged mining only available via getblock?

ahmedbodi commented 9 years ago

depends on how you implement it. if you implement the merged mining directly into the pool backend then you dont need getwork if youre going to do it seperately such as how most others do it then you need getwork