dapphub / ds-proxy

a proxy object that can compose transactions on owner's behalf
https://dapp.tools/dappsys/ds-proxy.html
GNU General Public License v3.0
311 stars 76 forks source link

Ds-Auth setOwner(msg.sender) causes DSProxyFactory to fail to deploy #3

Closed NiklasKunkel closed 7 years ago

NiklasKunkel commented 7 years ago

` contract DSProxyFactory { event Created(address sender, address proxy, address cache); mapping(address=>bool) public isProxy; DSProxyCache cache = new DSProxyCache();

  function build() returns (DSProxy) {
      var proxy = new DSProxy(cache);               //create new proxy contract
      Created(msg.sender, proxy, address(cache));   //trigger Created event
      proxy.setOwner(msg.sender);                   //set caller as owner of proxy
      isProxy[proxy] = true;                        //log proxys created by this factory
      return proxy;
  }

}`

If I comment out proxy.setOwner(msg.sender); the deployment works. Need to investigate if this is a problem with DS-Auth or a problem with the way I've tried to utilize it.

NiklasKunkel commented 7 years ago

Fixed in global_cache branch merge to master. Closing.