Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

SecretStore: ECDSA session for cases when 2*t < N #7739

Merged
merged 10 commits into from
Mar 1, 2018
Merged

Conversation

svyatonik
Copy link
Collaborator

@svyatonik svyatonik commented Jan 30, 2018

implements #7615
this PR is only about c0a8f8d commit

Background description is in the #7615

API changes

  1. Schnorr signature is generated by GET-ting /schnorr/{server_key_id}/{signature}/{message_hash} (previously: GET to /{server_key_id}/{signature}/{message_hash})
  2. ECDSA signature is generated by GET-ting /ecdsa/{server_key_id}/{signature}/{message_hash}

Usage example

  1. create account
  2. sign server key id using this account:
    curl --data-binary '{"jsonrpc": "2.0", "method": "secretstore_signRawHash", "params": ["0x00a329c0648769A73afAc7F9381E08FB43dBEA72", "", "0x0000000000000000000000000000000000000000000000000000000000000003"], "id":1 }' -H 'content-type: application/json' http://127.0.0.1:8545/
  3. create server key using generated signature:
    curl -v -X POST http://localhost:8082/shadow/0000000000000000000000000000000000000000000000000000000000000003/bd22d48735e5d711fffa03d9a987366acd50359a7a27f2fb0480e5339a44bf155b36ee1f2f860f4fcbb1171cf68dd59ac5f95d407137cafec4898f3e562fa9d700/1
  4. sign message hash using server key:
    curl -v http://localhost:8082/ecdsa/0000000000000000000000000000000000000000000000000000000000000003/bd22d48735e5d711fffa03d9a987366acd50359a7a27f2fb0480e5339a44bf155b36ee1f2f860f4fcbb1171cf68dd59ac5f95d407137cafec4898f3e562fa9d700/0000000000000000000000000000000000000000000000000000000000000005

@svyatonik svyatonik added A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. M4-core ⛓ Core client code / Rust. labels Jan 30, 2018
@5chdn 5chdn added this to the 1.10 milestone Jan 30, 2018
@svyatonik svyatonik added A0-pleasereview 🤓 Pull request needs code review. and removed A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. labels Jan 31, 2018
@debris debris added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Mar 1, 2018
@debris debris merged commit 0da6c7e into master Mar 1, 2018
@debris debris deleted the secretstore_ecdsa branch March 1, 2018 08:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-looksgood 🦄 Pull request is reviewed well. M4-core ⛓ Core client code / Rust.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants