revault / practical-revault

Version 0 specifications for a Revault deployment
Creative Commons Attribution 4.0 International
33 stars 9 forks source link

Batch encrypted signatures #71

Closed darosior closed 3 years ago

darosior commented 3 years ago

In https://github.com/re-vault/practical-revault/blob/master/messages.md#synchronisation-server ,

{
    "method": "sig",
    "params": {
        "pubkey": "Secp256k1 public key used to sign the transaction (hex)",
        "encrypted_signature": {
          "pubkey": "Curve25519 public key used to encrypt the signature",
          "signature": "base64-encoded encrypted Bitcoin ECDSA signature"
        },
        "id": "transaction txid"
    }
}

Should be

{
    "method": "sig",
    "params": {
        "pubkey": "Secp256k1 public key used to sign the transaction (hex)",
        "encrypted_signatures": {
          "Curve25519 public key used to encrypt the signature": "base64-encoded encrypted Bitcoin ECDSA signature"
        },
        "id": "transaction txid"
    }
}

If i'm not overlooking something

darosior commented 3 years ago

This can just be closed if we adopt #72