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

Check coin data is compatible with commitment #11

Closed
riemann89 opened this issue Mar 28, 2019 · 4 comments
Closed

Check coin data is compatible with commitment #11

riemann89 opened this issue Mar 28, 2019 · 4 comments
Labels
test Task related to the testing of the project

Comments

@riemann89
Copy link
Contributor

In testEtherMixing.py and testERCTokenMixing.py: when Charlie receives 1 ETH/1 ETHToken from BOB, he should

  • Recompute the commitment from the coin's data (to check the validity of the payment)

  • Make sure this recomputed commitment is equal to the commitment at address cm_address1BtB

@riemann89 riemann89 added the test Task related to the testing of the project label Mar 28, 2019
@rrtoledo
Copy link
Contributor

rrtoledo commented Jan 9, 2020

We have to be very careful here because this can easily lead to deanonymising a coin. This could be discussed in #144 .

My first ideas to avoid deanonymisation so would be to either:

  • pull the last commitments after a sufficiently large random delay (after receiving a coin) and check that our recomputed commitment is in the retrieved list
  • pull the last commitments at regular intervals (independent of receiving a coin)
  • have the Mix contract emit the new root and commitment

@dtebbs
Copy link
Contributor

dtebbs commented Jan 9, 2020

Should be addressed in fb90b95 (part of #130).
(Also discussed in #144)

@AntoineRondelet
Copy link
Contributor

AntoineRondelet commented Jan 21, 2020

@rrtoledo as mentioned by @dtebbs this has already been discussed here: #130 (comment)

@AntoineRondelet
Copy link
Contributor

Closing this issue since it has been addressed in #130

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test Task related to the testing of the project
Projects
None yet
Development

No branches or pull requests

4 participants