search

IrrigationProtocol / irrigation-contracts-diamond

MIT License
3 stars 1 forks source link

Modify Blacklist #24

Open SuperGNUS opened 1 year ago

SuperGNUS commented 1 year ago

In light of the new EU push for $1000 limit add a limit to the blacklist, which really becomes a grey list. Default initialization of zero will needs to mean unlimited, so we'll have to use a very small amount for blacklisting completely so probably 0x1 which is ver small in fixed point and probably cannot trade.

The values should be represented in WEI.

The front end then can get the geoip address and set this maximum trade value on converting to zether.

Highly important that we will need multiple range proofs and we will need to upgrade to something like Bulletproofs++ or new NIZK code that GNUS will use

ruymaster commented 1 year ago

Yes, range proofs are still needed. By the way, I think that checking geolocation is possible only off chain. Even though we will use ZKP, banning transfer by geolocation is not possible on chain, in my thought. I will research solution more.

SuperGNUS commented 1 year ago

Yes, just add a limit variable to the zether deposit function. The front end will calculate that limit with geoip and pass it in. No on chain is needed.

Front end will need some updatable table for GEOIPs and limits. And when doing a zether transaction will need to pass current WEI value of the asset as well