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

feat: Alert system #887

Closed
wants to merge 2 commits into from
Closed

feat: Alert system #887

wants to merge 2 commits into from

Conversation

jjyr
Copy link
Contributor

@jjyr jjyr commented May 23, 2019

Introduce:
https://en.bitcoin.it/wiki/Alert_system

Purpose:
Implement the alert system in CKB for urgent situation,
In CKB early stage we may meet the same crisis bugs that Bitcoin meet,
in urgent case, CKB core team can send an alert message across CKB P2P network,
the client will show the alert message, the other behaviors of CKB node will not change.

The Alert System will be removed soon once the CKB network is considered mature.

Implement:

  1. Add a new network protocol alt.
  2. Add a default disabled RPC send_alert.

@jjyr jjyr requested a review from a team May 23, 2019 07:44
@nervos-bot
Copy link

nervos-bot bot commented May 23, 2019

@quake is assigned as the chief reviewer

@nervos-bot
Copy link

nervos-bot bot commented May 23, 2019

CI status of the fork branch is Build Status

Copy link
Member

@janx janx left a comment

Choose a reason for hiding this comment

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

Will the node relay the alert message? Is the message signature checked before relay?

@@ -0,0 +1,26 @@
[package]
name = "ckb-alert-system"
Copy link
Member

Choose a reason for hiding this comment

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

ckb-network-alert is more appropriate.

util/multisig/Cargo.toml Show resolved Hide resolved
@jjyr
Copy link
Contributor Author

jjyr commented May 23, 2019

Will the node relay the alert message? Is the message signature checked before relay?

https://github.com/nervosnetwork/ckb/pull/887/files#diff-f9056d164d152ac4dad0a349994e8c30R138

All alert messages need to be verified with signatures before the relay.

@jjyr
Copy link
Contributor Author

jjyr commented May 23, 2019

I decide split multisig to another PR.

@jjyr jjyr closed this May 23, 2019
@jjyr jjyr mentioned this pull request May 23, 2019
@jjyr jjyr changed the title feat: Alert system feat: Network Alert May 28, 2019
@jjyr jjyr changed the title feat: Network Alert feat: Alert system May 28, 2019
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