Move code to check amount of tokens transferred to library #52 #13
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ezynda3
changed the title
0xDEnYO
added a commit
that referenced
this pull request
Jul 2, 2024
0xDEnYO
added a commit
that referenced
this pull request
Oct 23, 2024
…ssFacetPackedV3 v1.0.0, ReceiverAcrossV3 v1.0.0) (#687) * Implment and add tests for LibUtil::revertWith() * remove unneeded import * deploy to staging * temp - DO NOT PUSH * split facets, test files, deploy scripts etc. to V1/V3 * fix: replaces address parameter in facet * tests fixed * demoscript can do ERC20 and native bridging * ReceiverAcrossV3 + deploy script added (and deployed) * added dest calls on demoscript * ReceiverAcrossV3 updated (out-of-gas handling) & redeployed * ReceiverAcrossV3 updated (will revert for out-of-gas during swap exec) & redeployed * Successfully sent native+destCall with demoScript * simplified demo script to only use one fromChain * fixed failing test * fix spelling error in import * update forge coverage command in Github action * adds a check that ensures receiver addresses match in case of no dst call * solves stack-too-deep issue in GenericSwapFacetV3 test file (popped up due to solc upgrade) * adds destinationCallFlag validation * implements gas savings in ReceiverAcrossV3 and redeploys to staging * updates docs * removes coverage files as these should not be synced to github * gas optimization: use solady lib in ReceiverAcrossV3 * change min solc version to ^0.8.17 * removes native asset handling from receiver since this is not required for AcrossV3 * removes coverage files as these should not be synced to github * removes unused import * redeployed ReceiverAcrossV3 to arbitrum * updates CalldataVerificationFacet to support AcrossV3 * removes payable keyword (audit issue #13) * removes unused import (audit issue #4) * removes payable keyword from handleV3AcrossMessage function (audit issue #5) * removes unused import (audit issue #7) * adds more detailed explanations to revert handling in Receiver contract (audit issue #11) * narrows variable type (audit issue #9) * re-deploys facets to optimism staging * updates expected gas values in tests * trying to fix failing types action * undo last commit * updates typechain command in package.json * finally fixed $§%&$%§&$% CI issue * updates evm-version used for testing in Github action * updates evm-version used for testing in Github action #2 * removes forge coverage from github action (unused anyway) * disable PR reminder for PRs with label waitForBackend * redeployed AcrossFacetV3 and ReceiverAcrossV3 to OPT & POL staging * audit log updated and report added * Deploy AcrossV3 (except for zksync) * Deploy AcrossV3 packed (except for zksync) * Deploy to zksync * Deploy packed facet * add task to update approvals for AcrossFacetPackedV3 * Fix AcrossFacetPackedV3 script * update deploy requirements * Fix update scripts * Fix update scripts * Cleanup * fix imports * import paths updated for zkSync scripts * fixes one more import * adds ready_for_review triggers to git actions * adds support for exclusiveRelayer parameters * redeployed facets to arbitrum and optimism STAGING * update audit report (incl exclusivity relayer) * deployments to various networks * zksync deploy scripts updated * updates deploy script for AcrossFacetPackedV3 * proposed diamondCuts & periphery update to all relevant diamonds (except: zksync) * Deploy Across V3 to zkSync * update across demo scripts * Update logs for zksync * update coverage command in package.json * diamond logs updated * bump version of EmergencyPauseFacet * update pragma --------- Co-authored-by: Ed Zynda <ezynda3@gmail.com> Co-authored-by: Ed Zynda <ezynda3@users.noreply.github.com> Co-authored-by: Max Klenk <max@klenk.biz>
ezynda3
added a commit
that referenced
this pull request
Oct 28, 2024
* Implment and add tests for LibUtil::revertWith() * remove unneeded import * deploy to staging * temp - DO NOT PUSH * split facets, test files, deploy scripts etc. to V1/V3 * fix: replaces address parameter in facet * tests fixed * demoscript can do ERC20 and native bridging * ReceiverAcrossV3 + deploy script added (and deployed) * added dest calls on demoscript * ReceiverAcrossV3 updated (out-of-gas handling) & redeployed * ReceiverAcrossV3 updated (will revert for out-of-gas during swap exec) & redeployed * Successfully sent native+destCall with demoScript * simplified demo script to only use one fromChain * fixed failing test * fix spelling error in import * update forge coverage command in Github action * adds a check that ensures receiver addresses match in case of no dst call * solves stack-too-deep issue in GenericSwapFacetV3 test file (popped up due to solc upgrade) * adds destinationCallFlag validation * implements gas savings in ReceiverAcrossV3 and redeploys to staging * updates docs * removes coverage files as these should not be synced to github * gas optimization: use solady lib in ReceiverAcrossV3 * change min solc version to ^0.8.17 * removes native asset handling from receiver since this is not required for AcrossV3 * removes coverage files as these should not be synced to github * removes unused import * redeployed ReceiverAcrossV3 to arbitrum * updates CalldataVerificationFacet to support AcrossV3 * removes payable keyword (audit issue #13) * removes unused import (audit issue #4) * removes payable keyword from handleV3AcrossMessage function (audit issue #5) * removes unused import (audit issue #7) * adds more detailed explanations to revert handling in Receiver contract (audit issue #11) * narrows variable type (audit issue #9) * re-deploys facets to optimism staging * updates expected gas values in tests * trying to fix failing types action * undo last commit * updates typechain command in package.json * finally fixed $§%&$%§&$% CI issue * updates evm-version used for testing in Github action * updates evm-version used for testing in Github action #2 * removes forge coverage from github action (unused anyway) * disable PR reminder for PRs with label waitForBackend * redeployed AcrossFacetV3 and ReceiverAcrossV3 to OPT & POL staging * audit log updated and report added * Deploy AcrossV3 (except for zksync) * Deploy AcrossV3 packed (except for zksync) * Deploy to zksync * Deploy packed facet * add task to update approvals for AcrossFacetPackedV3 * Fix AcrossFacetPackedV3 script * update deploy requirements * Fix update scripts * Fix update scripts * Cleanup * fix imports * import paths updated for zkSync scripts * fixes one more import * adds ready_for_review triggers to git actions * adds support for exclusiveRelayer parameters * redeployed facets to arbitrum and optimism STAGING * update audit report (incl exclusivity relayer) * deployments to various networks * zksync deploy scripts updated * updates deploy script for AcrossFacetPackedV3 * proposed diamondCuts & periphery update to all relevant diamonds (except: zksync) * Deploy Across V3 to zkSync * update across demo scripts * Use foundry-zksync * Add missing floating pragmas and ignore tests and scripts when compiling for zksync * Use separate scripts for zksync * forge install: era-contracts * Fixes * Remove old scripts * Update verification and begin implementing create2 deployments * Try and predict address using system contract * Add remaining AcrossV3 deploy scripts * Add missing deploy scripts to zksync * Update logs * Update log for EmergencyPauseFacet * Auto build foundry-zksync docker image if it does not exist * Start adding special update scritps for zksync * Confirm deployment if already deployed * Redeploy EmergencyPauseFacet * Remove duplicates * Add comments and remove unfinished/incomplete scripts * Fix copy paste errors * Remove unused code * Update comments * Remove lcov dir * Add comment --------- Co-authored-by: Daniel Bläcker <d.blaecker@gmail.com> Co-authored-by: Max Klenk <max@klenk.biz>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.