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

BIP 310: Stratum protocol extensions #709

Open
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
3 participants
@honzik666

honzik666 commented Aug 1, 2018

Hi,

This proposal went formally through the mailing list: https://lists.linuxfoundation.org/pipermail/bitcoin-dev/2018-May/015978.html without any further discussion. Currently, this extension has been implemented by multiple pools. I would like to kindly ask for BIP # assignment and have this merged into upstream bips.

Best regards,

Jan Capek

@achow101

This comment has been minimized.

Show comment
Hide comment
@achow101

achow101 Aug 1, 2018

Member

As an aside, perhaps the stratum protocol itself should be specified in BIP documents as well so that people don't have to go to other sites to learn about the stratum protocol itself before reading the extensions BIP. The Stratum protocol has BIPs 40 and 41 but there are no documents for those present in the repo.

Member

achow101 commented Aug 1, 2018

As an aside, perhaps the stratum protocol itself should be specified in BIP documents as well so that people don't have to go to other sites to learn about the stratum protocol itself before reading the extensions BIP. The Stratum protocol has BIPs 40 and 41 but there are no documents for those present in the repo.

@honzik666

This comment has been minimized.

Show comment
Hide comment
@honzik666

honzik666 Aug 2, 2018

That is a good point, we can compile a stratum BIP from the spec that we published here: http://stratumprotocol.org/

honzik666 commented Aug 2, 2018

That is a good point, we can compile a stratum BIP from the spec that we published here: http://stratumprotocol.org/

@luke-jr

This comment has been minimized.

Show comment
Hide comment
@luke-jr

luke-jr Aug 9, 2018

Member

Please use BIP number 310.

Also, while this does address backward compatibility to a limited degree, it should ideally be discussed in a dedicated section of the document. Ideally, even if you insist on breaking compatibility with the established (and technically superior) mining.capabilities for some reason, it would be nice to document that incompatibility there as well.

Member

luke-jr commented Aug 9, 2018

Please use BIP number 310.

Also, while this does address backward compatibility to a limited degree, it should ideally be discussed in a dedicated section of the document. Ideally, even if you insist on breaking compatibility with the established (and technically superior) mining.capabilities for some reason, it would be nice to document that incompatibility there as well.

@luke-jr luke-jr added the New BIP label Aug 9, 2018

@luke-jr luke-jr changed the title from Import initial draft of stratum extension bip to BIP 310: Stratum protocol extensions Aug 9, 2018

@honzik666

This comment has been minimized.

Show comment
Hide comment
@honzik666

honzik666 Aug 9, 2018

honzik666 commented Aug 9, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment