51 Commits (f2cb415eecc7ba7600d7bd7faa66f65c54c182e7)

Author	SHA1	Message	Date
Anna Carroll	f2cb415eec	fix: Use low-level call instead of contract call (#439) ... * Use low-level call instead of contract call * Add typescript	3 years ago
James Prestwich	e6becba053	refactor: several rust improvements (#414) ... * refactor: several rust improvements - add partials to latest deploy - fix the prefix iterator in db - run the core updater loop in the current span - correct a processor bug that would prevent startup after any proofs were cached - correct a processor bug that would prevent startup with the initial tree * feature: root acceptability checks - expose acceptableRoot in Replica.sol - check if the root is acceptable before dispatching a proof from the processor	3 years ago
James Prestwich	6487fcb3eb	Processing Refactor (#409) ... * refactor: have proversync cache proofs upon tree update * refactor: start introducing acceptableRoot into replica * refactor: set confirmAt to 1 for initial root * refactor: remove previous from replica trait * refactor: processor no longer bails on missing proof * tests: fix the tests * refactor: remove last use of tokio::time::interval * refactor: remove redundant condition in replica	3 years ago
Anna Carroll	d20438434c	feat: RecoveryManager role on GovernanceRouter (#353) ... * RecoveryManager role Fix inequality * Require recovery activated to exit recovery * Add RecoveryManager tests * Move utils & pass lint * Test transfer RecoveryManager role: * Refactor RecoveryManager tests * Change block.number -> timestamp * ran prettier * Add recovery manager to typescript * Fix comment * Re-arrange inRecovery check Co-authored-by: Conner Swann <me@connerswann.me>	3 years ago
Anna Carroll	5b2e48a358	Remove return values from `handle` (#404) ... * Remove return values from handle * Add test for reverting handle function * package-lock * package-lock again * Add tests for various 'bad' recipients * lint: prettier Co-authored-by: James Prestwich <james@prestwi.ch>	3 years ago
James Prestwich	41b19b70dd	refactor: add prove_and_process to replicas enum	4 years ago
James Prestwich	76ab2034d8	refactor: fix the annoying process bug. misc rust updates	4 years ago
James Prestwich	11573866e9	refactor: nextToProcess is now a uint32	4 years ago
James Prestwich	f00674ca25	refactor: home 0-indexes sequences (#397) ... * refactor: home 0-indexes sequences * fix: sequence Co-authored-by: Erin Hales <mcbridee093@gmail.com>	4 years ago
James Prestwich	ebe13489af	feature: semaphores and durations	4 years ago
James Prestwich	a5e91cedf4	bug: disable external common initializer (#373) ... * bug: disable external common initializer * chore: remove old code and unused lint comment	4 years ago
Erin Hales	ed4cda0dc3	fix: lastProcessed -> nextToProcess (#371) ... * fix: change lastProcessed to nextToProcess * fix: bugs * refactor: test-all script * fix: delete file, initialIndex 0 * bug: fix solidity tests with incorrect vector keys Co-authored-by: James Prestwich <james@prestwi.ch>	4 years ago
Erin Hales	81546c6a2b	test: GovernanceRouter upgrade and setUpdater calls (#334) ... * add: initial upgrade test * debug: upgrade callMessage test and isGovernorRouter * add: pre upgrade tests * remove: getUpgradeBeacon func * fix: variable declarations * fix: bug in test setup * lint * test: gov router can call UpdaterManager to change the Updater on Home * fix: rebase on main and fix conflicts * refactor: remove contracts.upgradeBeacon * refactor: DRY code, formatCallData * refactor: remove unused function * refactor: _isGovernorRouter func * refactor: remove mockRecipient from formatCallData * refactor: callLocal instead of process through Replica * refactor: get dataLen for calls internally * refactor: callLocal and expect event * refactor: rename formatCall * test: cross-chain upgrade message * test: prove and process upgrade message * fix: cross-chain upgrade test * test: prove and process cross-chain message * enhance: deployMultipleChains function * debug: merkle proof * fix: message leaf * fix: format calls and merkle proof * fix: broken test	4 years ago
Luke Tchang	ae28e02316	test/bug: verify Rust/Solidity OpticsMessage and leaf equivalence (#363) ... * feature: outputs hardcoded merkle proof * fix: wrong sender address * debug: print sender receiver for StampedMessage * prog: leaf check failing * fix: fixes message pack ordering bug and outputs correct message and leaf json * fix: add message and leaf output to precommit fn * fix: use hex string instead of hardcoded str * feature: adds output proof test function	4 years ago
Luke Tchang	f750437766	test: adds ConnectionManager unit tests (#336) ... * test: adds owner enroll/unenroll tests * test: creates WalletProvider class to simplify getting unused wallets * test: adds unpermissioned unenrollment test case * test: adds unenrollReplica revert test cases * test: asserts isReplica returns T/F * refactor: dry up wallet * fix: removes isOwner typo	4 years ago
James Prestwich	fee2f4cd7a	feature: Home rejects messages larger than 2KiB body (#333)	4 years ago
Luke Tchang	9ebd31e086	fix: remove proveAndProcess dependency on test ordering (#317) ... * fix: optics message sender and recipients determined in test/utils.js rather than in test file * fix: sender and mockRecipient init * fix: uses singleton object rather than awaiting promise explicitly * fix: rename MockRecipient test contract TestRecipient to avoid confusion w/ actual mocks	4 years ago
Erin Hales	7cdd461a34	test: cross chain governance (#312) ... * fix: OpticsIdentifier as_ref casts ethereum address conditionally * feature: adds GovernanceRouter test boilerplate and fixes ConnectionManager test setup * test: adds testProcess and revert message utility * test: adds test for handle revert with unenrolled replica * test: adds test for reverting in handle on nonGovernorRouter message * rename: formattedMessage --> opticsMessage * prog: blocked on incorrect setting of governorDomain = localDomain * fix: fix after domainHash rebase * test: adds success case transfer governor test * test: adds setRouter message success case * prog: Call message success case failing due to datalen mismatches * prog: storing in dynamic struct array reverts silently * add: initial setup * add: transfer governorship test * add: signUpdate * refactor: combine bridgerouter tests * fix: cast to address, set sequence * delete: old gov router tests * Fix transfer governor test * fix: debug call storage, get sequence * fix: bad merge rust code * revert: stale changes * refactor: nits, create thirdDomain/thirdRouter vars * refactor: extract some code into formatOpticsMessage * refactor: formatOpticsMessage * refactor: deployOptics and devDeployOptics * fix: bug, clean up code * lint * fix: add numCalls (#315) * fix: initial stab at adding numCalls * debugging: numCall type conversion * fix: read _numCalls from bytes29 * use MSG_PREFIX_LEN * clean: remove unused imports * fix: data bug, refactor a bit * add: Transaction reverted silently * fix: bug in formatCalls * refactor: formatCalls * add: check ret value for testProcess from GovernorRouter * enhance: more explicit naming * rename: BYTE_LEN to NUM_ITEMS * slow down turbo * enhance: assign _numCalls variable Co-authored-by: Luke Tchang <ltchang@stanford.edu> Co-authored-by: anna-caroll <anna.s.carroll@gmail.com>	4 years ago
James Prestwich	7bfe50bd5f	Prestwich/watcher unenroll (#324) ... * feature: adds xapp interface enum to optics-base * feature: adds xapp manager to watcher and has watcher unenroll replicas * refactor: adds Vec<ConnectionManagers> and separates double update handling from unenrollment * fix: watcher-partial config takes vec of ConnectionManager configs * fix: updates comment and adds tracing instrumentation * fix: bad ConnectionManager setup reports error instead of unwrap * fix: concurrently awaits double_update and unenroll tasks * fix: actually awaits two vec of futures concurrently * fix: signed failure notification uses home domain (fixing conflation of replica/home domain) * fix: remove origin_domain after rebase * fix: rename double_update_res double_update * refactor: always use full-length identifier in failure notification Co-authored-by: Luke Tchang <ltchang@stanford.edu>	4 years ago
Anna Carroll	af35686796	Initializable Ownable on Home (#305) ... * Remove owner from Home and add transfer UpdaterManager to current UpdaterManager * Update comment	4 years ago
Erin Hales	676c495668	refactor: rename domains rust (#306) ... * refactor: nuke origin_domain * refactor: rename domain_hash * refactor: rename update local_domain to match solidity * refactor: update local_domain to domain * refactor: rename replica destination_domain * fix: remames origin in home/replica and converts back to domain_hash * fix: rust json testcase output field renames * fix: renames contract domain hash functions homeDomainHash * rename: update.domain --> update.homeDomain * fix: variable names and casing * fix: minor things, delete extra vector file * fix: homeDomainHash test loop * enhance: move common functionality to _signatureHash * fix: rename internal homeDomainHash func * fix: lint * fix: sucky merge * fix: bad rebase * fix: domain field accidentally got deleted * fix: overwrite instead of append for updating precommit script (#311) * fix: rename remaining 'origin' references * fix: use origin for Rust and Solidity formatted messages Co-authored-by: Luke Tchang <ltchang@stanford.edu>	4 years ago
Luke Tchang	ddea9d9764	fix: OpticsIdentifier conditionally casts ethereum address (#296) ... * fix: OpticsIdentifier as_ref casts ethereum address conditionally * fix: fixes TestXAppConnection manager name * fix: creates as_ref_local to preserve normal as_ref	4 years ago
Anna Carroll	f63926c7dc	polish: use Initializable.sol from OpenZeppelin (#303) ... * Initializable Common * Initializable Home * Initializable GovernanceRouter * Initializable Replica	4 years ago
Erin Hales	5f04a62cfb	fix: domainHash (#289) ... * fix: domainHash * refactor: rename signatureDomain * fix: add remoteDomain * refactor: rename signatureDomain * fix: remoteDomain/localDomain Replica tests * fix: domainHash view function * fix: signatureDomain tests * fix: recoverWatcherFromSig * refactor: localDomain immutable * fix: initialize_common	4 years ago
Anna Carroll	d72def136d	Fix comment (#291) ... * Fix comment * Fix comment	4 years ago
Anna Carroll	bf58066f67	Ensure events are emitted whenever Updater and UpdaterManager are set (#290)	4 years ago
Erin Hales	45fbc4f7f3	refactor: rename internal funcs (#282)	4 years ago
Erin Hales	9320e06533	refactor: reorder functions (#280)	4 years ago
Erin Hales	5b9c5c614a	Solidity Refactor (#277) ... * refactor: rename originDomain and ownDomain * refactor: create libs folder * refactor: rename interfaces * fix: common tests * fix: bad sig error * fix: event name * fix: update abi * fix: XAppConnectionManager tests	4 years ago
Luke Tchang	03fa9bcded	feature: adds XAppConnectionManager interface (#273)	4 years ago
Anna Carroll	34dce892fc	test: cross-chain messaging w/ deploy script (#262) ... * Test deploy script + cross-chain message passing * Replicas start with empty queues * Track the latestRoot for each chain * Fix: ownDomain vs. originDomain mixup in Replica * Rename test file * Refactor tests to make cross-chain utils reusable * Make return values and naming better * Moar improve * Smaller TODO * Update naming for clarity * Document crossChainTestUtils.js * Fix docs for formatMessage * Document deployCrossChainTest.js * Document generateTestChainConfigs * Document deployOptics script * Move types to appropriate directory * Document SimpleMessage.test.js * Remove double asterisk * Fix rebase * fix imports	4 years ago
Luke Tchang	94fd095043	refactor: renames UsingOptics contract XappConnectionManager (#258) ... * refactor: renames UsingOptics contract XappConnectionManager * fix: Xapp --> XApp	4 years ago
Erin Hales	f004e58131	refactor: sortition --> updater manager (#226) ... * add: basic updater manager * add: setUpdater is ownable * refactor: remove sortition, update UpdateManager * fix: function calls * rename: current function to currentUpdater * refactor: Common doesn't inherit from UpdaterManager * add: permissioned setUpdater function on Home * add: TODO * refactor: move onlyUpdaterManager modifier below initialize * refactor: rename TestSortition * fix: updater.sol -> updaterManager.sol, pass in _updaterManager NOT _updater * remove unused import * refactor: updaterManager on Home, updaterManager contract does not set home on initialization * fix: call Home's setUpdater, get updaterManager's address * enhance: emit NewUpdater event * enhance: emit NewHome event, docs * enhance: emit UpdaterSlashed event * add: Governable contract * documentation: add dev note * fix: bad merge, TestUpdaterManager * fix: goodbye sortition * add: deploy updaterManager * fix: pass in updater, delete TestUpdaterManager * refactor: more explicit naming * refactor: currentUpdater to updater getter * add: isContract check, indexed params * refactor: delete Governable contract * rename: _updaterAddress	4 years ago
Luke Tchang	dc52d01882	feature: adds watcher support for UsingOptics unenrollment (#229) ... * fix: only takes domain and updater * fix: unenrollReplica reverts instead of silent failure and events emitted * feature: adds failure notification structs * feature: adds signer to watcher and produces signed failure notifications on fraud * fix: update after rebasing on main with watcher disk changes * fix: removes stale db_paths from updater and watcher * prog: adds test function in optics-core to produce signed updates for common test suite (broken) * fix: renames previousRoot oldRoot and fixes originDomain to match Common tests * test: adds test to verify watcher SignedFailureNotifications in Solidity * refactor: adds and uses Addresses enum to support 20 and 32 byte addresses * refactor: make an OpticsIdentifier type * feature: add function to cast identifier to ethereum * fix: fixes replica deployment in UsingOptics tests Co-authored-by: James Prestwich <prestwich@clabs.co>	4 years ago
Anna Carroll	769d6b1c28	Script to Deploy Contract Suite (#230) ... * Move GovernanceRouter to optics-core * Update deploy upgrade scripts * Add system deploy script * Fix issues with deploy script * set home on updaterManager * Fix booogs * chore: remove governance from workflow file Co-authored-by: James Prestwich <prestwich@clabs.co>	4 years ago
Luke Tchang	1490207c7d	feature: adds unrenroll permissioning to UsingOptics (#219) ... * feature: adds permissioned unenrollReplica to UsingOptics * feature: adds setWatcherPermission for owner * fix: add check that replica updater matches signed * fix: rename watcherPermissions * fix: only takes domain and updater * fix: adds domainHash to watcher signed data * refactor: recovers watcher from sig and checks permissions after * fix: unenrollReplica reverts instead of silent failure and events emitted	4 years ago
Anna Carroll	be4704e85a	Implement Controller (#224) ... * Minimal Controller * Remove deploy functionality from UpgradeBeacon - not necessary interface * Update tests to use controller to upgrade * fix rebase	4 years ago
Anna Carroll	53f79d9964	Migrate from `constructor` to `initialize()` (#195) ... * Refactor constructors to initialize * Fix merge, fix lint * Move ownDomain to initialize; logically paired with instance, not implementation * Add initialize to Common * Solidity tests passing * Migrate GovernanceRouter to initialize * Fix GovernanceRouter * Integrate Replica and ProcessingReplica; fix function collision in abigen * Fix: rust syntax; remove commented line * Gate initialize() on state.UNINITIALIZED * Rename queue.init() -> queue.initialize()	4 years ago
Anna Carroll	f44e60a86a	refactor: interfaces directories (#157) ... * Rename Types -> BridgeMessage; Router -> BridgeRouter * Add BridgeTokenInterface.sol * Add MessageRecipientInterface.sol * Add SortitionInterface.sol * nit: rename Interface to I * nit: rename Interface to I Co-authored-by: Erin Hales <mcbridee093@gmail.com>	4 years ago
Luke Tchang	f7d2a6cc7f	refactor: changes UsingOptics to contract reference (#170) ... * refactor: changes UsingOptics to contract ref * feature: adds setUsingOptics to bridge and gov * fix: fix after rebase * refactor: constructor takes usingOptics address * fix: adds ownable() to token registry constructor	4 years ago
Luke Tchang	398f928cbc	docs: adds natspec documentation to core contracts (#137) ... * docs: adds documentation to Home contract * docs: specifies what destinationAndSequence calculation gives * docs: adds documentation for Replica contract * docs: adds documentation to Common * docs: adds documentation to Queue * docs: adds documentation to Merkle	4 years ago
James Prestwich	3d4aa756d1	feature: a simple script to make a test deploy of a home contract (#150) ... * feature: a simple script to make a test deploy of a home contract * bug: fix madison gasprice	4 years ago
Erin Hales	19d24ea096	clean: delete old comments (#131)	4 years ago
Luke Tchang	074a34c85d	test: increases core contract test coverage (#113) ... * test: adds tests for empty queues and ensures proveAndProcess reverts * test: ensures invalid double update does not fail home * test: adds queueLength helpers to TestReplica and TestHome * test: ensures queue dequeue many reverts if number exceeds length * test: adds test for batch updates to Home * test: adds successful proveAndProcess test case * fix: replaces assert statements with expect	4 years ago
Erin Hales	c6126431be	WIP/enhancement: linting and coverage (#106) ... * enhancement: linting * enhancement: pre-commit hooks with cargo-husky * enhancement: pre-commit hooks for solidity * fix: pre-commit script * ok to add package-lock.jsons * add: npm husky * format: README * add: bypass info to README * clean: remove husky script * configure: play nice, prettier and eslint * fix: lint settings * refactor: ethlint to solhint * lint: compiler version * lint: everything * lint: optics-bridge * lint: mark visibility of state * lint: TokenRegistry * enhancement: make pre-commit.sh executable * chore: clean code and dependencies * lint: optics-core tests * enhance: pre-commit.sh * fix: delete commented code, kathy/cargo.toml	4 years ago
Luke Tchang	2fbbcf7fc3	test/refactor: adds test for destinationAndSequence calculations + refactors domainHash test file creation (#110) ... * refactor: domain_hash util in optics-core outputs json file instead of printing hashes * test: adds Rust destinationAndSequence test to output vector * test: adds test case to Home tests for comparing Rust and Solidity destinationAndSequence calculations * fix: replaces & with | and fixes destinationAndSequence js utility * fix: removes #[test] tags on test case output functions * fix: asserts destinationAndSequence function args are capped at u32::MAX	4 years ago
Luke Tchang	7b2d9d7762	test: tests Dispatch event data and refactors enqueue message helper (#93) ... * feature: adds messageToLeaf and calcDestinationAndSequence utils * feature: adds currentLeafIndex getter to TestHome * test: adds test that checks Dispatch event and refactors enqueue message helper * fix: fixes package-lock.json * feature: adds dispatch event lookup util function on optics.Home * fix: renames TestHome function nextLeafIndex instead of currentLeafIndex	4 years ago
Luke Tchang	b6da94665f	fix: require status of Message.None in process() (#99) ... * fix: require status of Message.None in process() * fix: change err message * test: adds test for trying to prove same message again	4 years ago
James Prestwich	0b78a672ae	chore: fix the tests James broke -_-	4 years ago
Luke Tchang	2dd857bbb8	feature: adds untested ProverSync struct to optics-core (#78) ... * feature: basic ProverSync setup * prog: basic setup for linking enqueuings to roots * refactor: modifies Dispatch event to contain treeSize field and condenses destination and sequence into single field * feature: adds leaf_by_tree_size methods to Home abi and logic for updating local tree in prover_sync * format: runs cargo fmt * prog: comments out unused code from previous design * fix: replaces slip44 references with domain * refactor: moves interval to parameter instead of ProverSync struct field * fix: deletes commented out code from NewLeaf design * refactor: modifies ProverSync error handling to bubble up ProverSyncError type * fix: removes slip44 reference * refactor: adds incremental merkle tree to ProverSync and batch updates prover on successful incremental update * fmt: runs cargo fmt * fix: fixes raw_message_by_sequence filtering after destination and sequence event fields were combined to single field in Home * fix: simplifies error handling, adds retry counter for failed api calls, and clones incremental merkle * fix: renames leaves_by_tree_size to leaves_by_tree_index * fix/docs: fixes Home Dispatch event to use leafIndex instead of treeSize and fixes docs to reflect change * fix: changes num_retries to u32 * refactor: cleans up error handling * docs: adds comment about why leafIndex is count() - 1 * fix: deletes old NewLeaf event from Home * fmt: runs cargo fmt * fix: removes num_retries field * refactor: simplifies destination_and_sequence util * fix: fixes typo in warning message * refactor: derives transparent errors * docs: improves comments for prover-sync/incremental merkle behavior * fix: restarts poll_updates loop if prover root was updated while we were building leaf vector from incremental merkle * feature: adds ProverSync::new function * fix: ProverSync stops polling updates on invalid local_root * fmt: runs cargo fmt	4 years ago