pegnet / pegnetd

The pegnet daemon to track txs, conversions, etc
Other
13 stars 14 forks source link

fat-103 signing timestamp randomizer #107

Open WhoSoup opened 4 years ago

WhoSoup commented 4 years ago

Crossposting this PR: https://github.com/Factom-Asset-Tokens/fatd/pull/52

Once this bug is fixed in the fatd code, we'll need to update pegnetd's import in order to pull in the changes. This affects someone who sends multiple identical transactions via the cli in the same second. The timestamp would be the same due to the bug and they would count as replay transactions.

Emyrk commented 4 years ago

The branch eth_gateway has the latest fat103 updates. It has all the dep updates. We are already working on a fork of /factom and I don't see our fork joining the original anytime soon. So we should just fix this in our fork.

WhoSoup commented 4 years ago

So we should just fix this in our fork.

That is also a possibility!