Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Draft EIP: BLS12-381 Key Generation #2333

Merged
merged 1 commit into from
Nov 25, 2019
Merged

Conversation

CarlBeek
Copy link
Contributor

@CarlBeek CarlBeek commented Oct 31, 2019

This is the proposed standard for BLS12-381 Key Generation for use within Eth2 as well as by the larger blockchain industry. It has existed in its own repo for a while now, but this is a better place to enshrine it as a standard for now.

Copy link
Member

@axic axic left a comment

Choose a reason for hiding this comment

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

I think this satisfies the requirements for a draft.

@axic
Copy link
Member

axic commented Nov 7, 2019

@CarlBeek is it OK to merge or do you expect any changes soon?

@axic
Copy link
Member

axic commented Nov 22, 2019

@CarlBeek ping

Adds discussions-to

Fix typos

Remove "www"

Apply @MrChico's suggestions from code review

Co-Authored-By: MrChico <martin.lundfall@gmail.com>

Switch to EIP numbering

Tree idx < 2**320

Requires seed >= 128 bits

Actually switch to salt to uint32

Drop TS/JS implementation
@CarlBeek
Copy link
Contributor Author

@axic

I have squashed the commits, so everything is ready to be merged here.

@axic
Copy link
Member

axic commented Nov 25, 2019

Thanks! No need to squash though, because we merge with squash merges anyway 😉

@axic axic merged commit 783fc89 into ethereum:master Nov 25, 2019
CarlBeek added a commit to CarlBeek/EIPs that referenced this pull request Nov 26, 2019
…tore

* 'bls_keystore' of github.com:CarlBeek/EIPs: (47 commits)
  fix link to heading
  Fix spelling
  Fix email address
  Draft EIP: BLS12-381 Deterministic Account Hierarchy (ethereum#2334)
  Fix some URLs and require 2333 too
  Add name to metadata title (ethereum#2370)
  Draft: BLS12-381 Key Generation (ethereum#2333)
  Automatically merged updates to draft EIP(s)  (ethereum#2397)
  Hard fork proposal to address the Ice age (ethereum#2387)
  Automatically merged updates to draft EIP(s) 1767 (ethereum#2262)
  EIP-2021: Payoutable Token (ethereum#2021)
  EIP-2009: Compliance Service (ethereum#2009)
  EIP-2019: Fundable Token (ethereum#2019)
  Use solidity/javascript highlighting in various EIPs (ethereum#2372)
  EIP-2018: Clearable Token (ethereum#2018)
  EIP-1996: Holdable token (ethereum#1996)
  Fix the username of @pizza-r0b in EIP-2309 (ethereum#2389)
  Clarify that empty accounts also return 0 in EIP-1052 (ethereum#2388)
  dType Functions Extension - Decentralized Type System for EVM (ethereum#2267)
  Fix spelling of GitHub [R4R] (ethereum#2369)
  ...
tkstanczak pushed a commit to tkstanczak/EIPs that referenced this pull request Nov 7, 2020
Apply @MrChico's suggestions from code review

Co-Authored-By: MrChico <martin.lundfall@gmail.com>
Arachnid pushed a commit to Arachnid/EIPs that referenced this pull request Mar 6, 2021
Apply @MrChico's suggestions from code review

Co-Authored-By: MrChico <martin.lundfall@gmail.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants