mimblewimble / grin-wallet

Grin Wallet
Apache License 2.0
183 stars 133 forks source link

payment proof verification broken in Owner API V3 #667

Closed marekyggdrasil closed 1 year ago

marekyggdrasil commented 1 year ago

Describe the bug Generating proof (ok), verifying proof in CLI (ok), verifying proof in Owner API (broken)

To Reproduce Steps to reproduce the behavior:

  1. Send transaction to one of your wallets using SRS flow
  2. Export payment proof grin-wallet export_proof -i 13 example_proof.json
  3. Verify this proof grin-wallet verify_proof example_proof.json, if you do it soon you will see error that excess not found on chain
  4. Verify the same proof using Owner API, you will get PaymentProof: Invalid recipient signature

Expected behavior The response from Owner API should be consisted with the response from CLI

Screenshots N/A

Desktop (please complete the following information):

OS

$ sw_vers
ProductName:    Mac OS X
ProductVersion: 10.15.7
BuildVersion:   19H1715

wallet

$ grin-wallet -V
grin-wallet 5.1.0

Additional context N/A

marekyggdrasil commented 1 year ago

Sorry, it is my bad, I have been running wrong script that had hardcoded values and it was always giving same responses.