79 Commits (f0a18a16bf8e9c7e23b7c8334f6da45ba9c442ba)

Author	SHA1	Message	Date
Anna Carroll	567631e510	Remove nextToProcess from Replica (#629) ... * Remove nextToProcess from Replica * Get tests passing * package-lock * refactor: remove next_to_process from rust Co-authored-by: James Prestwich <james@prestwi.ch>	3 years ago
ember arlynx	5f041c9db1	drive-by cleanups (#592) ... * reorganize rust code into dirs * rust: ignore vendor dir * how did kathy go missing? * drive-by cleanups: abstraction in update_abis, hyperlink, shell scripts are mostly shellcheck clean * rust readme: note agents dir * cargo fmt --all * remove stray comment * cargo update * why am i fighting my computer rn * patch over sloppy rebase * shore up test vector file handling * clarify how to add a new chain * add rust-toolchain.toml for 1.54 * DRY abis_dir * chore: add clippy installation to CI * github actions will use rust-toolchain.yaml now * move rust-toolchain so the action will find it rustup seems to scan upwards for this file, should pick it up fine when using from within * use obsolete rust-toolchain file location, because the action is naive * chore: specify toolchain in rust.yml * chore: specify toolchain in rust.yml in all workflows * chore: return clippy to rust.yml Co-authored-by: ember arlynx <ember@lunar.software> Co-authored-by: James Prestwich <james@prestwi.ch>	3 years ago
James Prestwich	d48ef53f22	feature: de-verbose logging by adding explicit display impls	3 years ago
James Prestwich	0c7c7041a5	refactor: improve layout of optics-core & bug: no eip-155 for updates	3 years ago
James Prestwich	c576771bac	chore: bump ethers to use legacy feature in optics-core	3 years ago
James Prestwich	13644d5957	KMS CLI (#547) ... * feature: sweep bin * refactor: hardcode gas req for simple sweep tx * refactor: update to latest version of ethers * feature: kms cli * bug: short designator conflict + docs * bug: fix value parsing * chore: temp fix for simple sends * refactor: delete deprecated kms bins, roll features into kms-cli	3 years ago
James Prestwich	7b501442b1	refactor: add aws signer (#516) ... * refactor: add aws signer * lint: clippy and fmt * lint: fix unsequenced reads * feature: add a small bin to print key address	3 years ago
Erin Hales	9c9fb87b03	test: migrate tests and deploy process to ts (#400) ... * refactor: migrate js lib to ts * refactor: rewrite extendEnvironment in ts * add: sample tests * add: extend hre * refactor: ts deploy and signer type * refactor: devDeploy for tests * document: isTestDeploy param, add common.test.ts * add: common tests * fix: common tests * clean: remove sample script * refactor: remove Signer type and use ethers.Signer * refactor: typescript directory * build: generate typechain * refactor: typescript code (#399) * refactor: optics-tests/optics-deploy * fix: install typechain * fix * fix: ts errors * add: merkle tests (#417) * add: merkle tests * clean: remove commented code * fix: merkle tests * test: add queue tests (#422) * Add queue tests * fix: add await to fix nonce bug * test: home ts (#401) * add: home tests * add: home tests * add: deploy home (broken) * refactor: add num confirmations to config * refactor: clean up around ts tests (#424) - add testChain.ts with utility functions for making test Chain and Deploy - refactor the Optics hardhat extension - improve typing of enums in Optics lib - remove most references to waffle (prefer ethers) - remove isTestDeploy from deployment args in favor of a test? on Deploys * refactor: clean test deploy, home tests passing * refactor: deployOptics * fix: white space * lint: fix white space * refactor: testCase vectors and imports * fix: rust test vector generation * fix: missing await Co-authored-by: James Prestwich <10149425+prestwich@users.noreply.github.com> * test: add message test (#423) * test: add message test * fix: replace require with import * fix: import for queue.test.ts * enhance: add type check * fix: import conflicts * test: add upgrade tests (#443) * test: add upgrade tests * fix: missing arg * refactor: clean up code * remove: unused code * refactor: deploy proxy * fix: unsaved code * fix: ts conflicts * add: utils ts (#453) * draft * refactor: remove waffle from home tests * fix: types, remove waffle Co-authored-by: yoduyodu <wang7ong@gmail.com> * test: add XAppConnectionManager tests (#456) * test: add XAppConnectionManager tests * test: initial setup for XAppConnectionManager tests * test: most XAppConnectionManager tests passing * fix: failing test * test: SimpleMessage ts (#468) * test: add cross-chain test utils * test: simpleMessage, mostly passing * fix: prove and process test * test: add initial state test, clean up * test: recoverymanager ts (#473) * test: add recoveryManager initial tests * fix: set recoveryManager in ts deploy * fix: set governor * test: replica ts (#483) * test: add replica tests * debug: fix failing tests * WIP: test: governance router/upgrade ts (#470) * test: add governance router tests * debug: some tests * debug: Gov Router tests * refactor: upgradeUtils * fix: missing await * test: clean up ts tests (#484) * clean: imports * refactor: utils * refactor: remove logs during testing * fix: weird bug, ethers.getSigners messes up describe blocks * delete: solidity/optics-core/js * update: pre-commit script for ts * update: pre-commit and scripts * fix: test, update with main * clean: types * fix: bad recipient handle * add: todo for gov router test * add: add back verify deploy stuff in js for now * bug: fix the governance upgrade test (#490) * fix: gov router upgrade test * feature: use TS in both solidity packages * bug: install deps in typescript dir in tests * feature: enhanced github action install * chore: disable automerge, and move lint before test Co-authored-by: Tong Wang <wang7ong@gmail.com> Co-authored-by: James Prestwich <10149425+prestwich@users.noreply.github.com> Co-authored-by: James Prestwich <james@prestwi.ch>	3 years ago
James Prestwich	a38f4f5153	chore: bump ethers to main in all cargo.toml files	3 years ago
James Prestwich	aff52ce2a9	bug: tracing and pending txn fixes ... - use branch of ethers-rs with pendingtx poll delay - use tracing::Instrumented in most agent trait methods - add in_current_span to most spawns - add an id generator to jaeger tracing	3 years ago
James Prestwich	5722a698f2	Refactor tracing subscriber instatiation (#466) ... * WIP(broken): start breaking out tracing config * WIP: custom abstraction over log formats * WIP(feat): clean jaeger instantiation in Subscriber layers * refactor: improve jaeger layer instantiation and application. Add filter layer * lint: self by value on copy type * feature: implement Jaeger configuration * bug: specify reqwest collector client and remove doc fragment * refactor: use std convert traits where possible * refactor: introduce DroppedError	3 years ago
James Prestwich	1593d3b0c9	feature: time out txns that have been pending confirmation for 10m (#457)	3 years ago
James Prestwich	56139b1715	bug: remove leftover dbg	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
James Prestwich	76ab2034d8	refactor: fix the annoying process bug. misc rust updates	3 years ago
James Prestwich	ac58c6a1cb	feature: MessageStatus	4 years ago
James Prestwich	7013122170	bug: incorrect deserialization of messages	4 years ago
James Prestwich	bb2768384a	bugs: lots of rust agent bugs	4 years ago
James Prestwich	191a9fefac	bug: fix processor error on unupdated system (#387)	4 years ago
James Prestwich	d99f15c041	bug: configuration bugs wrt config and hexstring	4 years ago
James Prestwich	31539b786a	Prestwich/hex string rust (#378) ... * feature: hex string for configuration * chore: test vectors * chore: remove dbg prints * chore: cleanup import and extra line * chore: remove unused dep	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
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	2eac18dd6e	chore: delete extra lockfiles in core and base (#346)	4 years ago
Luke Tchang	c4b34fb2df	test: adds mock ConnectionManager interface + watcher failure test cases (#314)	4 years ago
Luke Tchang	5ce1430ae0	refactor: move destinationAndSequence to optics-core and run output in precommit (#326) ... * refactor: moves destinationAndSequence to optics-core and adds output function to precommit * fix: domainHash output function writes to homeDomainHashTestCases.json	4 years ago
Luke Tchang	befde049a6	fix: update signer to support EIP-155 (#329) ... * fix: provider with SignerMiddleware udpates chain_id of signer to match provider * fix: add ethers wallets feature flags to optics-core cargo.toml	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
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	59efd7c10b	feature: rust output functions conditionally run in precommit (#309) ... * feature: adds rust output bins to precommit * fix: compiles abis first and formats json * fix: json formatting * fix: use git diff-index for json * fix: git add abis if updated	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
Luke Tchang	03fa9bcded	feature: adds XAppConnectionManager interface (#273)	4 years ago
Luke Tchang	94fd095043	refactor: renames UsingOptics contract XappConnectionManager (#258) ... * refactor: renames UsingOptics contract XappConnectionManager * fix: Xapp --> XApp	4 years ago
Luke Tchang	72d6ae20fb	feature: replaces Signer generic with enum (#246) ... * fix: only takes domain and updater * fix: unenrollReplica reverts instead of silent failure and events emitted * feature: adds failure notification structs * fix: update after rebasing on main with watcher disk changes * fix: removes stale db_paths from updater and watcher * 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: adds signer enum variants * feature: fixes signer enum and adds to updater and watcher * fix: adds SignersError enum to accomodate LedgerError * fix: removes Addresses enum after rebase * fix: deletes duplicate impls after rebase * fix: uses ? instead of map_err Co-authored-by: James Prestwich <prestwich@clabs.co>	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
Luke Tchang	73b989285e	feature/refactor: saves/loads processor prover-tree to/from disk + fixes abstraction layers between prover and merkle (#189) ... * refactor: joins two chain api calls and spawns task for wait and check * feature: adds disk persistence to updater and checks for conflicting updates * feature: gets db_path from settings block * feature: adds open_db util and replaces in updater * feature: adds encode/decode to optics-core lib structs * test: adds test_util func to run db tests in closure * feature: adds serde * feature: proversync saves new leaves to disk * feature: adds loading prover from disk * fix: replaces ne_bytes to be_bytes * rebase: removes duplication after rebase on updater-history branch * fix: removes bincode * refactor: fixes abstraction layers between prover and merkle variants * refactor: uses prefix_iterator in prover * fix: has rocksdb imports draw from github * refactor: adds UsingPersistence trait and adds to updater, prover, proversync * refactor: update UsingPersistence to use Encode + Decode * feature: iterator in UsingPersistence Co-authored-by: James Prestwich <james@prestwi.ch>	4 years ago
Luke Tchang	b9694211f4	feature: adds updater history and persists to disk (#180) ... * refactor: joins two chain api calls and spawns task for wait and check * refactor: poll_and_handle_update returns join handle * feature: adds disk persistence to updater and checks for conflicting updates * feature: gets db_path from settings block * feature: adds open_db util and replaces in updater * feature: adds encode/decode to optics-core lib structs * test: adds test_util func to run db tests in closure * fix: uses Encode::to_vec	4 years ago
Luke Tchang	a1040a04e2	feature: adds configurable pause to updater (#167) ... * feature: adds pause to updater and fixes tests to account for change * refactor: joins two chain api calls and spawns task for wait and check * refactor: poll_and_handle_update returns join handle	4 years ago
Luke Tchang	1cdc8c8614	test: updater tests (#152) ... * prog: adds Homes type to optics-base and has AgentCore take in Arc<Homes> * refactor: separates prover and prover_sync; all agents compiling except watcher * refactor: adds EthereumReplica variant and fixes agents * refactor: adds MockReplica variant to Replicas * fmt: runs cargo fmt * test: adds updater poll_and_handle_update test * fix: removes duplicates after bad rebase * test: adds no update path test to updater test * fix: uses cfg_attr to only warn missing docs in test	4 years ago
Luke Tchang	9cae2f469a	refactor: contract trait refactor + prover/prover_sync separation (#151) ... * prog: adds Homes type to optics-base and has AgentCore take in Arc<Homes> * refactor: separates prover and prover_sync; all agents compiling except watcher * refactor: adds EthereumReplica variant and fixes agents * refactor: adds MockReplica variant to Replicas * fmt: runs cargo fmt * feature: adds From implementations for Homes and Replicas * feature: adds From<EthereumHome> for Homes * feature: adds From<EthereumReplica> for Replicas * feature: adds Other variant to Homes and Replicas * lint: use Box in all homes and replicas to satisfy clippy * bug: fix Ethereum handling in ChainSetup::try_into_ functions Co-authored-by: James Prestwich <james@prestwi.ch>	4 years ago
Luke Tchang	01b6aa43ea	test: adds optics-test crate and unit test for watcher::check_double_update (#138) ... * refactor: extracts double update check to helper * prog: mockall derive debug on Common and Home broken * feature: MockHomeContract in optics-base * test: adds test for watcher UpdateHandler check_double_update * feature: adds replica mock contract * refactor: moves mocks to new optics-test crate * fix: satisfy clippy gods * fix: removes dev dependencies from optics-base * fix: fixes optics-test docstring and unecessary assert_eq Co-authored-by: James Prestwich <prestwich@clabs.co>	4 years ago
James Prestwich	6e80bfde3e	chore: appease the clippy gods	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
Anna Carroll	69f64b04f8	feature: add untested Processor daemon (#97) ... * feature: add untested Processor daemon * feature: add an untested processor daemon * docs: clean up and clarify the watcher * bug: fix watcher loop conflating leaf index and sequence * feature: improved handling for main watcher loop - info logging on loop resets due to acceptable faults - returns an error if the local prover is out of sync with the home chain Co-authored-by: James Prestwich <prestwich@clabs.co>	4 years ago
Luke Tchang	0d5fbbe23b	feature/WIP: watcher checks for fraudulent updates (#88) ... * feature: separates watch into watch_replica and watch_home and checks for fraudulent updates for replica * docs: adds documentation for watcher behavior * todo: adds todo! macro * docs: updates watcher description to include fraudulent updates * fix: has watcher check for fraudulent updates by submitting to home * refactor: has check_double_update return error and handles in calling scope * refactor: improve watcher reporting of double updates Co-authored-by: James Prestwich <prestwich@clabs.co>	4 years ago
Erin Hales	5d3668495a	enhancement: move abi files to shared folder (#94) ... * moved abis to shared folder * basic (surely flawed) optics ethereum package * bug: fix circular imports, rearrange files and paths, lint * fix: unused import * fix: serde Deserialize * fix test issues, delete settings * enhancement: separated out home and replica abis * fix: pub use statements * cargo update and lint * fix: doc code snippet, trim dependencies	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
Luke Tchang	c57b9ded00	feature: adds untested Watcher daemon (#73) ... * enhance: adds Update event emission to Replica and signed_update_by_old root to Replica abi * feature: adds core run logic for watcher observing single replica * feature: adds watch_home logic and run_many override * refactor: moves signed_update_by_old_root to common * refactor: uses Common enum for Home/Replica * refactor: removes enum and uses ?Sized to remove Sized constraint * fix: removes unnecessary import * fix: runs cargo fmt and replaces references to origin_slip44 with domain * fix: fixes deadlock setup for history read/write * fix: fixes poll_signed_update to use signed_update_by_new_root * fix: fixes off-by-one error for topics for signed_update abi methods * docs: fixes comment formatting on signed_update methods	4 years ago
Luke Tchang	99b650d495	refactor: separates full from partial messages with Message vs StampedMessage (#77) ... * refactor: separates Message from DispatchedMessage * refactor: changes DispatchedMessage to StampedMessage * fix: ran cargo fmt	4 years ago