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

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.

EIPS/eip-draft_bls_keygen.md Outdated Show resolved Hide resolved
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 <[email protected]>

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 <[email protected]>
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 <[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