Giveth / milestonetracker-ui

User Interface for the Milestone Tracker
GNU General Public License v3.0
8 stars 11 forks source link

UI Should validate contract Bytecode during init. #3

Open barryWhiteHat opened 7 years ago

barryWhiteHat commented 7 years ago

When a contract is imported it should be confirmed that the contracts byte code is on a white list of allowed values.

I am worried about people creating malicious contracts to attack our client side user interface. Tricking people into pointing our user interface at them. This is probably impossible right now but its a good practice to use from the outset in my opinion.

I raise it here as opposed to making PRs because

  1. I don't know where all the contracts are imported.
  2. I want to get your opinions on it before i move ahead with it.