Skip to content

Conversation

@harding
Copy link
Collaborator

@harding harding commented Oct 22, 2018

A bit wordy and technical this week; I'll try to be more concise and simple next week.

For the nmap results, I'm happy to email or IRC the commands to any of you who want to replicate the test (I just don't like posting exact port scanning commands publicly in order to reduce abuse).

@harding
Copy link
Collaborator Author

harding commented Oct 22, 2018

Oh, Travis appears to be not running, perhaps due to the GitHub data inconsistency incident. I can confirm that commit ee3c0dc passes all tests for me locally.

you've set a custom RPC port or otherwise have enabled a customized
configuration.

A [PR][Bitcoin Core #14531] has been opened to Bitcoin Core to make
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is the correct PR to reference: bitcoin/bitcoin#14532

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

D'oh, I thought I checked all the links. Thanks! Fixed (force-pushed).

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I still think it might be broken as the link is missing? This is what I see when I test it locally: "A [PR][Bitcoin Core #14532] has"

@harding harding force-pushed the 2018-10-23-newsletter branch from ee3c0dc to 2feedbf Compare October 22, 2018 20:07
Copy link
Contributor

@jnewbery jnewbery left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK 2feedbf.

A few tiny nits. Happy to publish without any changes though.

creating the pubkey, they may also make it possible for fewer than all
of them to sign, e.g. 2-of-3 of them must cooperate to sign. This can
be much more efficient than Bitcoin's current multisig, which requires
placing *m* signatures and *n* pubkeys into transactions for m-of-n
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have a slight preference for k-of-n, because m-of-n sounds very similar to n-of-n when spoken. I'm aware that both variants are commonly used, so if you prefer m-of-n that's also fine.

for ECDSA means it also supports pure ECDSA multiparty schemes as
well. No changes are required to the consensus rules, the P2P
protocol, address formats, or any other shared resource. All you
need are two wallets that both implement multiparty ECDSA key
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

'two or more wallets that implement' ?

evaluate their security properties, and consider implementing
them---and some experts are already busy working on implementing a
consensus change proposal that would enable a Schnorr signature
scheme that would also provide for multiparty pubkeys and signatures
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

suggest that we emphasize that Schnorr threshold signatures are far less complex to implement. Something like: 'that would provide for a far less complex scheme for multiparty pubkeys and signatures'

scheme that would also provide for multiparty pubkeys and signatures
(and which also provides multiple other benefits).

- [Fast Multiparty Threshold ECDSA with Fast Trustless Setup][mpecdsa goldfeder] by Rosario Gennaro and Steven Goldfeder
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

curious why [mpecdsa goldfeder] and not [mpecdsa gennaro]?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Haha, you're right, it's an incongruity. I was looking at the URL when choosing the link name and the paper is hosted on StevenGoldfeder.com. :-)

@jnewbery
Copy link
Contributor

ACK 0efd859

@jnewbery jnewbery merged commit 72204d9 into bitcoinops:master Oct 23, 2018
bitschmidty pushed a commit that referenced this pull request Oct 12, 2023
* newsletter270zh

* Update _posts/zh/newsletters/2023-09-27-newsletter.md

Co-authored-by: freeyao <[email protected]>

* Update _posts/zh/newsletters/2023-09-27-newsletter.md

Co-authored-by: freeyao <[email protected]>

* fix style

* fix style

---------

Co-authored-by: editor-Ajian <[email protected]>
Co-authored-by: freeyao <[email protected]>
Co-authored-by: Zhiwei(Jeffrey) Hu <[email protected]>
harding pushed a commit to harding/bitcoinops.github.io that referenced this pull request Sep 27, 2024
* newsletter270zh

* Update _posts/zh/newsletters/2023-09-27-newsletter.md

Co-authored-by: freeyao <[email protected]>

* Update _posts/zh/newsletters/2023-09-27-newsletter.md

Co-authored-by: freeyao <[email protected]>

* fix style

* fix style

---------

Co-authored-by: editor-Ajian <[email protected]>
Co-authored-by: freeyao <[email protected]>
Co-authored-by: Zhiwei(Jeffrey) Hu <[email protected]>
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 this pull request may close these issues.

3 participants