vmware / concord-bft

Concord byzantine fault tolerant state machine replication library
377 stars 147 forks source link

Changes to handle invalid coin(s) in privacy-wallet-service. #3002

Open snehal-das opened 1 year ago

snehal-das commented 1 year ago
snehal-das commented 1 year ago

Changes to the privacy-wallet-service is complete and tested to work as expected. Changing status from draft to ready for merge.

snehal-das commented 1 year ago

I am not able to edit the reviewers list, the only option I see is to mention the person here.

@bsenthilr, @yontyon Please take a look and add your thoughts.

Some items pending are:

  1. Add a proper error response instead of sending back an error string.
  2. Correct the initialisation of invalidCoinsInTransfer.
  3. Look for better options for error reporting.