Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update pool func visibility #1283

Merged
merged 8 commits into from
Aug 12, 2024
Merged

update pool func visibility #1283

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
ae4bd54
update pool funcs
matYang Aug 12, 2024
ffc9e4d
add internal versions of l/r in USDC pool
matYang Aug 12, 2024
11788d5
missed changes
matYang Aug 12, 2024
af97f0d
wrappers
matYang Aug 12, 2024
cf4c209
missed override
matYang Aug 12, 2024
c2e9b97
revert to using public
matYang Aug 12, 2024
62c4219
wrappers
matYang Aug 12, 2024
99b1d43
Merge branch 'ccip-develop' into pool-function-viz
matYang Aug 12, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion contracts/src/v0.8/ccip/pools/TokenPool.sol
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -157,7 +157,7 @@ abstract contract TokenPool is IPoolV1, OwnerIsCreator {
/// @param lockOrBurnIn The input to validate.
/// @dev This function should always be called before executing a lock or burn. Not doing so would allow
/// for various exploits.
function _validateLockOrBurn(Pool.LockOrBurnInV1 memory lockOrBurnIn) internal {
function _validateLockOrBurn(Pool.LockOrBurnInV1 memory lockOrBurnIn) internal virtual {
if (!isSupportedToken(lockOrBurnIn.localToken)) revert InvalidToken(lockOrBurnIn.localToken);
if (IRMN(i_rmnProxy).isCursed(bytes16(uint128(lockOrBurnIn.remoteChainSelector)))) revert CursedByRMN();
_checkAllowList(lockOrBurnIn.originalSender);
Expand Down
4 changes: 2 additions & 2 deletions contracts/src/v0.8/ccip/pools/USDC/USDCTokenPool.sol
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -97,7 +97,7 @@ contract USDCTokenPool is TokenPool, ITypeAndVersion {
/// @dev emits ITokenMessenger.DepositForBurn
/// @dev Assumes caller has validated destinationReceiver
function lockOrBurn(Pool.LockOrBurnInV1 calldata lockOrBurnIn)
external
public
virtual
override
returns (Pool.LockOrBurnOutV1 memory)
Expand Down Expand Up @@ -139,7 +139,7 @@ contract USDCTokenPool is TokenPool, ITypeAndVersion {
/// non-reverting offchainTokenData that can be supplied is a valid attestation for the
/// specific message that was sent on source.
function releaseOrMint(Pool.ReleaseOrMintInV1 calldata releaseOrMintIn)
external
public
virtual
override
returns (Pool.ReleaseOrMintOutV1 memory)
Expand Down
2 changes: 1 addition & 1 deletion contracts/src/v0.8/ccip/test/helpers/MultiTokenPool.sol
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -157,7 +157,7 @@ abstract contract MultiTokenPool is IPoolV1, OwnerIsCreator {
/// @param lockOrBurnIn The input to validate.
/// @dev This function should always be called before executing a lock or burn. Not doing so would allow
/// for various exploits.
function _validateLockOrBurn(Pool.LockOrBurnInV1 memory lockOrBurnIn) internal {
function _validateLockOrBurn(Pool.LockOrBurnInV1 memory lockOrBurnIn) internal virtual {
if (!isSupportedToken(lockOrBurnIn.localToken)) revert InvalidToken(lockOrBurnIn.localToken);
if (IRMN(i_rmnProxy).isCursed(bytes16(uint128(lockOrBurnIn.remoteChainSelector)))) revert CursedByRMN();
_checkAllowList(lockOrBurnIn.originalSender);
Expand Down
Loading