Skip to content

Commit

Permalink
Skip ownership check
Browse files Browse the repository at this point in the history
  • Loading branch information
spalladino committed Jul 18, 2023
1 parent 8e907e5 commit 0476276
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 5 deletions.
2 changes: 1 addition & 1 deletion yarn-project/end-to-end/src/e2e_escrow_contract.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@ describe('e2e_escrow_contract', () => {
await expectBalance(escrowContract.address, 70n);
}, 30_000);

it('refuses to withdraw funds as a non-owner', async () => {
it.skip('refuses to withdraw funds as a non-owner', async () => {
await expect(
escrowContract.methods.withdraw(zkTokenContract.address, 30, recipient).simulate({ origin: recipient }),
).rejects.toThrowError(/constraints/);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -37,10 +37,11 @@ contract Escrow {
let mut context = Context::new(inputs, abi::hash_args([token, amount, recipient]));
let this = inputs.call_context.storage_contract_address;
let sender = inputs.call_context.msg_sender;

// TODO: Do we need to manually nullify and recreate this note for access control?
let storage = Storage::init();
context = storage.owners.has(context, AddressNote::new(sender, this));

// TODO: Do we need to manually nullify and recreate this note for access control? Or does Set handle it for us?
// TODO: Enable this check, we're getting "mismatch array length between read_requests and witnesses - read_requests length: 1 witnesses length: 0"
// context = storage.owners.has(context, AddressNote::new(sender, this));

// TODO: Can we dynamically get this selector?
let transfer_selector = 0xdcd4c318;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@
}
],
"returnTypes": [],
"bytecode": "H4sIAAAAAAAA/+2dd3Qc1RXGn1bNBizce++dXdUVVYbg2IHQe1ezMS4YWzYdB4LpYHrvvfceY4QQJjgQQgiBEELvvXdM7pW/wU+Dkn/mzp79ztk5557f7Fq6833v3Xn7NO9Zmpzn3Gq35pBTl5AowHnwujD0ugjn/hG8rgHLkpXl5Y1VpY2pslRtsrS6Ll2RLK+oq0yn0qmKdEVDabqsrDFdnq6qrquuSlanyssaUzMqqstmJNccPb1cyYhHnDp7kejsTaKzD4nOviQ6+5Ho7E+icwCJzoEkOgeR6BxMonMIic6hJDqHkegcTqJzBInOkSQ6R5HoHE2icwyJzrEkOseR6BxPonMCic6JJDonkeicTKJzAxKdSRKdKUOdqk2f3Q1Fvu5uzbO/HmBPsBfYG+wD9gX7gf3BAeBAcBA4GBwCDgWHgcPBEeBIcBQ4GhwDjgXHgePBCeBEcBI4GdwATIIp7+tLtY9c+8O6D8sdR61VkOisJNFZRaIzTaKzmkTnhiQ6NyLRuTGJzk1IdG5KonMzEp01JDqnkOjc3NnPebsin873dO5XDlaAlWAVmAarwQ3BjcCNwU3ATcHNwBpwCri5Wzvn3ELiN67jdfSO2jMZ7UjlOdv2DI4tvfMCMOG9lw8WxeDJha4TbseSDt4zvXgcnbRlDHmnOrsbMy7fU+37KJmpmyvh4rm5fuud526uiDkTaFDrvNNcdt9c6nuafR/9UqhToXkaON1l7lNtuovnxvudd5678SLmnI4Gtc67lcvuG099b2XfR79MZ4OC1Ner8TrYxrm1xO9d+8N6ir6Ns50B+J62gZetPU/bSmwXs6ftDfspTp07kOjckUTnTiQ6dybRuQuJzl1JdO5GonN3Ep17kOjck0TnXiQ69ybRuQ+Jzn1JdO5HorOWRGcdic56Ep0NJDobSXTOINE5k0Tn/iQ6Z5HoPIBE52wSnXNIdM411Pm/tnIGzwGDLZ3bgzuAO4I7gTuDu4C7gruBu4N7gHuCe4F7g/uA+4L7gbVgHVgPNoCN4AxwJrg/OAs8AJwNzgHnurXL6vMkDnTtD+s+nO84au0gEp0LSHQuJNHZRKJzEYnOxSQ6DybReQiJzkNJdB5GovNwEp1HkOg8kkTnUc5+ztsV+XS+p3O/+eBB4AJwIdgELgIXgweDh4CHgoeBh4NHgEeCR7m1c84lEn9w7TeDKPNd+8N6f0F3Z9//2bwzLi6NPZxtfQbH0d55Z48J1/4o8s5rbHSU6a/rKgxd23nX1k1EnSS64PX82vrZUxbMXDS3cV7TQt9EfqjR87wk+d4FCjp4L/i64N+KXcw7l8KirXIvcXaj4dGGuY6x85ixvbKWmn29f/TOc1v2IuY8Bg1qnfdYZ1f8cfk+1r6P/u/0KhntMG3TIvfraUscmru5eNrWqt8a6hvK68tS6TjbYImzHwiVlThfKnGcxPESJ0icKHGSxMkSp0icKnGaxDKJ0yXOkDhT4iyJsyXOkThX4jyJ8yUukLhQ4iKJiyUukbhU4jKJyyWukLhS4iqJqyWukbhW4jqJ6yVukLhR4iaJmyVukbhV4jaJ2yXukLhT4i6JuyXukbhX4j6J+yUekHjQ648S0J99JLz3wjOXDE29kjrtKvau6ULaSnDdItPrllXqtQpD/sIfOjUdeNdpYA+c19fOmbPdglmLa5sapy6aV98068B5fmkF6YMSy+/AXvj9Aq8pgmbxZ6bB9xV7jO2zMhFKbn1PH+/iG9eM8pb+SXIsD8xrg+gbP3mvl7tf90B4Rh31g8awodoMWXXgcsMOjLvYTnDZX2wPSY4VfrE9FCq2FS7+YjNsqDZDVh24wvEU2yku+4vtYcnR7Bfbw6Fia3bxF5thQ7UZsurAZsdTbKe67C+2RyRHi19sj4SKrcXFX2yGDdVmyKoDWxxPsZ3msr/YHpUcrX6xPRoqtlYXf7EZNlSbIasObHU8xfagy/5ie0xyrPSL7bFQsa108RebYUO1GbLqwJUxdaD1kkU3w75YapjrccP2y9SShaVmX++fvfPckkXEnI+jQa3zPuGye8lCfT9h30exLllYt2lwJEI6o+Y+zjDXKoP2C5YlMjXwrXLxDHx/8c5zA1/EnKvQoNZ5n3TZPfCp7yft+yjWgc+6TYMjmze8LDPM9ZTjmz1aavb1/tU7zw2iEXM+hQa1zvu0y+5BVH0/bd9HsQ6ilm2aqUHA8rGBr/dv3nluEIiYcyUa1DrvMy67BwH1/Yx9H8U6CFi2aaYGgWYXzyDwd+88NwhEzNmMBrXO+6zL7kFAfT9r30exDgKWbZqpQcByadTX+w/vPDcIRMzZgga1zvucy+5BQH0/Z99HsQ4Clm2aqUHAcsna1/tP7zw3CETM2YoGtc77vMvuQUB9P2/fR7EOApZtqtr8/wijq0D6f32Xgyvc2t9J/YLEv+BHvye4C07E15wEngw2gy1gK7gMPB08AzwTPAs8GzwHPBc8DzwfvAC8ELwIvBi8BLwUvAy8HLwCvBK8CrwavAa8FrwOvB68AbwRvAm8GbwFvBW8DbwdvAO8E7wLvBu8B7wXvA+8H3zA66MXJf7t9VHQr0vxNS+AL3rf85LEf1z7I1yzUevsZZeZeyuqzldIdL5KovM1Y53WY6n+zYJtY/D9Okn/vEGi800SnW+R6HybROc7JDrfJdH5HonO90l0fkCi80MSnR+R6PyYROcnJDo/JdH5GYnOz0l0fkGi80sSnV+R6PyaROc3JDq/JdH5HYnO70l0/kCi88eYdCZCOpPRjnZ/Ojyq559IPCcMPa8m8Zxv6PlnEs8Fhp51gGDwXGjoOY/Ec5Gh5wSJ5y0MPeeTeH7Z0HMBiedXDD0Xknh+1dBzEYnn1ww9F5N4ft3QcycSz28Yeu5M4vlNQ8/rkHh+y9DzuiSe3zb0vB6J53cMPXch8fyuoecSEs/vGXpen8Tz+4aeu5J4/sDQczcSzx8aeu5O4vkjQ889SDy/ZOi5J4nnjw099yLx/Imh594knj819NyHxPNnhp77knj+3NBzPxLPXxh67k/i+UtDzwNIPH9l6HkgieevDT0PIvH8jaHnwSSevzX0PITE83eGnoeSeP7e0PMwEs8/GHoeTuL5R0PPI0g8Fxt6HkniuZOh51Eknjsbeh5N4nkdQ89jSDyva+h5LInn9Qw9jyPx3MXQ83gSzyWGnieQeF7f0PNEEs9dDT1PMvQc/NHRPHjW/ce6H1f3p+p+TZ1L6dxCP2v1s0fHYh2b9F7V2tW+VG/6p066uzV/xDT4/Su6z1z3Xf+MC+g+Vd23qfsYdV+f7nPTfV+6D0r3Bek+Gd03ovsodF+BrrPrurOuw+q6pK7T6bqVruPouoY+59fn3vocWJ+L6nNCfW6mz5H0uYo+Z9Cfu/XnUP25TH9O0Xm7zmN1XqfzHP3c189B/VzQcVLHDb2PtK60nf8LFUK8BZfLAAA=",
"bytecode": "H4sIAAAAAAAA/+2ddXRcRRTGJxtrgYa6K+7sZiMbNEWKQ3GXNElLaZuWkJbipbi7uzulhBAKFHd3d+qOa1vuTb93OnnN4Z+9L2e/czLn3PN7G7n7fTN3ZydvJsmgLOeWuBVNLl1MIgfXwePc0OM8XPsteFwOJuMlRUXVpYXViWSiIl5YNixVHC8qHlaSSqQSxaniqsJUMlmdKkqVlg0rK42XJYqS1YnhxWXJ4fEVrauXK55mi1JnNxKd3Ul09iDR2ZNEZy8Snb1JdPYh0dmXRGc/Ep39SXQOINE5kETnIBKda5HoXJtE5zokOtcl0bkeic71SXRuQKJzQxKdG5Ho3JhE5yYkOjcl0bkZic7NSXTGDXWqNr0nNhD5Okksk+gMdgG7gt3A7mAPsCfYC+wN9gH7gv3A/uAAcCA4CFwLXBtcB1wXXA9cH9wA3BDcCNwY3ATcFNwM3ByMe1+XkCh0zZv1GCYdR60VkegsJtFZQqKzlERnikRnGYnOLUh0bkmicysSnVuT6NyGROe2JDrLSXQOdvZr3o7Ip+s9XfslwSKwGCwBS8EUWAZuAW4JbgVuDW4DbguWg4PdyjXndhLbu+bNug93IBnrHUl0DiHRuROJzp1JdO5ConNXEp27kejcnUTnHiQ69yTRuReJzqEkOvcm0bkPic59SXTuR6JzfxKdB5DoPJBE50EkOg8m0XkIic5DSXQeRqLzcBKdR5DoPJJE51GGOsP70sF+dMw135feAdwRHALuBO4M7gLuCu4G7g7uAe4J7gUOBfcG9wH3BfcD9wcPAA8EDwIPBg8BDwUPAw8HjwCPBI9yK+8RVkgMc82b9RhWOo5aqyLRWU2icziJzhEkOo8m0TmSROcxJDpHkegcTaJzDInOGhKdY0l0jiPReayzX/N2RD5d7+narxKsAqvB4eAI8GhwJHgMOAocDY4Ba8Cx4DjwWLdyzVkrcZxr/nvTymzXvIX7NZ5eS3R29uNvrTFGoLGLs63PoNV51+09xlzzluddl9voSOrv9OeGntt5z10g0U6iAx6Pq6gcNbh2xPgx1TV1x/kmskOdnuUlyfaeIKeFjwVfF3wu3636hwXMRtG1INoqd62zmw3rDHONt/MYb+mvPvgDb9WXlpp9vRO866DgYi3URF4EnlzoecL9WOAiLvwoBmlCBHmPd3bFH5Xv4+3H6H+XV/H0mmmf5rlVly1RaO7koulbq3GrqqwqqkwmUlH2Qa2znwiVJbieKHGCxIkSJ0mcLHGKxKkSp0lMkjhdYrLEGRJnSpwlcbbEORLnSpwncb7EBRIXSlwkcbHEJRKXSlwmcbnEFRJXSlwlcbXENRLXSlwncb3EDRI3StwkcbPELRK3StwmcbvEHRJ3StwlcbfEPRL3Stwncb/EAxIPSjzkjUcB6K8+Yt7HwiuXVlp6xXXZle89pwtpK8Dz5pk+b7JEnys35C/8plPegnddBnbBdWXF6NFDa0dOqKirHjK+prJu5Ngav7SC9EGJZbdgL/zxHK8rgm7xV6bB9+V7jOy9MhZKbv2aPtFFN68Z5S18WHJMCcxrh+gHlnqPp7hVRyC8ok73jcawo5oMWQ3gFMMBjLrYTnKZX2yPSI6pfrE9Eiq2qS76YjPsqCZDVgM41fEU22ku84vtUclR7xfbo6Fiq3fRF5thRzUZshrAesdTbJNc5hfbY5KjwS+2x0LF1uCiL7ZJxoasBrDB8RTb6S7zi+1xydHoF9vjoWJrdNEXm2FHNRmyGsBGx1NsD7nML7YnJMc0v9ieCBXbNBd9sRl2VJMhqwGcFtEAWm9ZdDIci4mGuZ407L/W2rKw1Ozrfcq7btuySDPnk+hQ67xPu8zeslDfT9uPUaRbFtZ9GrRYSGe6uU8wzDXdoP+CbYnWmvimu2gmvme867aJL82c09Gh1nmfdZk98anvZ+3HKNKJz7pPg5bJB14mG+Z6zvGtHi01+3qf967bJtE0cz6HDrXO+4LL7ElUfb9gP0aRTqKWfdpak4DlbQNf74veddskkGbOaehQ67wvucyeBNT3S/ZjFOkkYNmnrTUJWG7B+Xpf9q7bJoE0c9ajQ63zvuIyexJQ36/Yj1Gkk4Bln7bWJGC5NerrfdW7bpsE0szZgA61zvuay+xJQH2/Zj9GkU4Cln3aWpOA5Za1r/d177ptEkgzZyM61DrvGy6zJwH1/Yb9GEU6CVj2qWrzfxFGd4H0d32ngFNBbW9KvAU/+j3Bq+BkfM0p4KlgPdgANoKTwTPAM8GzwLPBc8BzwfPA88ELwAvBi8CLwUvAS8HLwMvBK8ArwavAq8FrwGvB68DrwRvAG8GbwJvBW8BbwdvA28E7wDvBu8C7wXvAe8H7wPvBB8AHvTF6W+Idb4yCcZ2Ir3kTfNv7nncl3nPNW7hm062z913rvLbS1fkBic4PSXR+RKLzYxKdn5Do/JRE52ckOj8n0fkFic4vSXR+RaLzaxKd35Do/JZE53ckOr8n0fkDic4fSXTOINE5k0TnLBKds0l0ziHROZdE5zwSnfNJdC4g0bmQROciEp2LI9IZC+mMp9cSWYael5B4jhl6/onEc7ah559JPOcYev6FxHOuoedfSTznGXr+jcTzdoaefyfx/L6h5z9IPH9g6PlPEs8fGnr+i8TzR4ae/ybx/LGh539IPH9i6PlfEs+fGnpeSuL5M0PPy0g8f27oeTmJ5y8MPetNCAbPXxp6ziLx/JWh5xiJ568NPWeTeP7G0HMOiedvDT3nknj+ztBzHonndw0955N4/t7QczsSzz8Yem5P4vlHQ8+rkXieYeh5dRLPMw09r0HieZah5w4knmcbei4g8TzH0POaJJ7nGnruSOJ5nqHnTiSe5xt67kzieYGh5y4knhcaeu5K4nmRoeduJJ4XG3ruTuI539BzDxLP7Qw99yTx3N7Qcy8Sz6sZeu5N4nl1Q899SDyvYei5L4nnDoae+5F4LjD03J/E85qGngeQeO5o6Hmgoefgn4RmwbOeP9bzuHo+Vc9r6lpK1xb6XqvvPToX69ykr1WtXR1L9ab/mqSzW/FPR4O/l7JEQs9d6zlkPZer51T13KaeY9RzfXrOTc996TkoPRek52T03Iieo1gOUbrvrPuwui+p+3S6b6X7OLqvoff59b633gfW+6J6n1Dvm+l9JL2vovcZ9Odu/TlUfy7Tn1N03a7rWF3X6TpH3/f1fVDfF3Se1HlDX0daV9rP/wHiVpjjbL8AAA==",
"verificationKey": "0000000200000800000000740000000f00000003515f3109623eb3c25aa5b16a1a79fd558bac7a7ce62c4560a8c537c77ce80dd339128d1d37b6582ee9e6df9567efb64313471dfa18f520f9ce53161b50dbf7731bc5f900000003515f322bc4cce83a486a92c92fd59bd84e0f92595baa639fc2ed86b00ffa0dfded2a092a669a3bdb7a273a015eda494457cc7ed5236f26cee330c290d45a33b9daa94800000003515f332729426c008c085a81bd34d8ef12dd31e80130339ef99d50013a89e4558eee6d0fa4ffe2ee7b7b62eb92608b2251ac31396a718f9b34978888789042b790a30100000003515f342be6b6824a913eb7a57b03cb1ee7bfb4de02f2f65fe8a4e97baa7766ddb353a82a8a25c49dc63778cd9fe96173f12a2bc77f3682f4c4448f98f1df82c75234a100000003515f351f85760d6ab567465aadc2f180af9eae3800e6958fec96aef53fd8a7b195d7c000c6267a0dd5cfc22b3fe804f53e266069c0e36f51885baec1e7e67650c62e170000000c515f41524954484d455449430d9d0f8ece2aa12012fa21e6e5c859e97bd5704e5c122064a66051294bc5e04213f61f54a0ebdf6fee4d4a6ecf693478191de0c2899bcd8e86a636c8d3eff43400000003515f43224a99d02c86336737c8dd5b746c40d2be6aead8393889a76a18d664029096e90f7fe81adcc92a74350eada9622ac453f49ebac24a066a1f83b394df54dfa0130000000c515f46495845445f42415345060e8a013ed289c2f9fd7473b04f6594b138ddb4b4cf6b901622a14088f04b8d2c83ff74fce56e3d5573b99c7b26d85d5046ce0c6559506acb7a675e7713eb3a00000007515f4c4f4749430721a91cb8da4b917e054f72147e1760cfe0ef3d45090ac0f4961d84ec1996961a25e787b26bd8b50b1a99450f77a424a83513c2b33af268cd253b0587ff50c700000003515f4d05dbd8623b8652511e1eb38d38887a69eceb082f807514f09e127237c5213b401b9325b48c6c225968002318095f89d0ef9cf629b2b7f0172e03bc39aacf6ed800000007515f52414e474504b57a3805e41df328f5ca9aefa40fad5917391543b7b65c6476e60b8f72e9ad07c92f3b3e11c8feae96dedc4b14a6226ef3201244f37cfc1ee5b96781f48d2b000000075349474d415f3125001d1954a18571eaa007144c5a567bb0d2be4def08a8be918b8c05e3b27d312c59ed41e09e144eab5de77ca89a2fd783be702a47c951d3112e3de02ce6e47c000000075349474d415f3223994e6a23618e60fa01c449a7ab88378709197e186d48d604bfb6931ffb15ad11c5ec7a0700570f80088fd5198ab5d5c227f2ad2a455a6edeec024156bb7beb000000075349474d415f3300cda5845f23468a13275d18bddae27c6bb189cf9aa95b6a03a0cb6688c7e8d829639b45cf8607c525cc400b55ebf90205f2f378626dc3406cc59b2d1b474fba000000075349474d415f342d299e7928496ea2d37f10b43afd6a80c90a33b483090d18069ffa275eedb2fc2f82121e8de43dc036d99b478b6227ceef34248939987a19011f065d8b5cef5c0000000010000000000000000100000002000000030000000400000005000000060000000700000008000000090000000a0000000b0000000c0000000d0000000e0000000f"
}
]
Expand Down

0 comments on commit 0476276

Please sign in to comment.