-
Notifications
You must be signed in to change notification settings - Fork 756
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.
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
BEP-174: Cross Chain Relayer Management #205
Merged
Merged
Changes from 3 commits
Commits
Show all changes
96 commits
Select commit
Hold shift + click to select a range
1d9c1e2
Add new proposed functions skeleton
emailtovamos 9c9423a
admin struct & removeAdminAddress code
emailtovamos 646ba57
remove, register, add admin + onlyAdmin modifier
emailtovamos 2747fcb
add relayer functions
emailtovamos 0065836
helper function to avoid repeat of code
emailtovamos 981c225
verifyRelayer and relayerExistsMap
emailtovamos d31bc5d
updateParam changes
emailtovamos 7513b3f
Remove unwanted code and variables
emailtovamos 38224e5
Remove old code
emailtovamos 5a5bb13
Formatting + efficient return for verify function
emailtovamos a40d34c
decouple registered and nonregistered admins, remove relayer before d…
emailtovamos 744819c
add old modifier checks before adding relayer
emailtovamos 0994f27
seed with current whitelists.todo: add relayers
emailtovamos 1f1135c
Refactor: replace admin with manager
emailtovamos 3efc715
contract and exist checks inside function
emailtovamos e808819
remove onlyWhitelabelRelayer for the most part.
emailtovamos 0fc3985
Remove register unregister from template file
emailtovamos ef4e086
isRelayer, not verifyRelayer
emailtovamos 9304110
isContract check for manager
emailtovamos 7f5751e
override in isRelayer
emailtovamos 5b757c7
refactor naming
emailtovamos ea38c55
addRelayer -> editRelayer
emailtovamos 9a6385c
remove most compilation errors apart from transfer
emailtovamos ba4c474
compilation error fix: payable+map
emailtovamos c3cdc8d
extra payables
emailtovamos 5d67e91
modify templates
MatusKysel 5ee2dc0
fix bugs in contract
MatusKysel a32ce75
Merge pull request #219 from bnb-chain/bep-174
emailtovamos 0bf58d1
Merge develop into relayer-admin-add-foundry
emailtovamos 711e76e
Merge pull request #220 from bnb-chain/relayer-admin-add-foundry
emailtovamos 5c501c4
Remove unneeded modifier + properly remove relayer
emailtovamos ce2ea8f
updateRelayer event and no need for removeRelayer
emailtovamos 6dfa169
For smooth transition and let old relayers exit
emailtovamos 21fd9d6
Only use updateRelayerEvent + Refacoring functions
emailtovamos b570b67
Better naming of some maps
emailtovamos dacd6ec
update only once
emailtovamos 744320e
Remove repeated event
emailtovamos 2820ea4
Remove onlyAllowedParty + correct var order
emailtovamos 627bb00
remove deposit + bring back old withdraw
emailtovamos 58a2a09
Merge pull request #225 from bnb-chain/relayer-admin-remove-100bnbdep…
emailtovamos 14f9aad
test for adding new param which doesn't work
emailtovamos 0ad173a
correct the test
emailtovamos e3c3829
Merge pull request #226 from bnb-chain/relayer-admin-foundrytest
emailtovamos af5b457
testunregister
emailtovamos f0b4590
Check emit event
emailtovamos 96d1b63
remove manager and ismanger, isrelayer test
emailtovamos 7c5f2a0
removeManagerByHimself test + comments
emailtovamos 466688d
testCurrentRelayerTransition() and update()
emailtovamos 61cc96c
Merge pull request #227 from bnb-chain/relayer-admin-foundrytest
emailtovamos d0b9e2a
Remove manager struct
emailtovamos 58f82eb
Refactor + remove unwanted functions
emailtovamos 673f929
Remove check that is not needed
emailtovamos 001e9b3
Refactor to less confusing name
emailtovamos 286f4ce
Ensure manager isn't already registered
emailtovamos ee74731
bug info
emailtovamos e941958
Test for relayer address 0 which had a bug
emailtovamos 07efff4
Remove todo which is already done now
emailtovamos 77c89cc
Merge pull request #228 from bnb-chain/relayer-admin-bugfix
emailtovamos 1e4ab68
Combine to one manager struct
emailtovamos 876f9ef
Remove dues
emailtovamos 3d05cbf
remove unwanted code
emailtovamos e03d020
Merge pull request #230 from bnb-chain/relayer-admin-removedues
emailtovamos 6186657
ensure 0 address is not a relayer
emailtovamos 164c20a
remove deposit and dues code in updateParam
emailtovamos d1c4947
template file update + correct whitelist mainnet addr
emailtovamos 1ebc8fd
check contract first
emailtovamos 919c2bb
refactor
emailtovamos ae24743
Refactor even names and regenerate interface
emailtovamos 8055814
payable manager not required
emailtovamos 195eccb
bug fix of twice adding of same relayer + test
emailtovamos 8ad0ad7
emit events in initialisation as well
emailtovamos ba6aac5
update the template file based on recent changes
emailtovamos 06927b4
public to internal + remove unused event
emailtovamos 0785d1d
Test to ensure a contract can't be a relayer
emailtovamos c9298af
2 step relayer registration. todo: test update
emailtovamos 15380dc
Fix testAddManager
emailtovamos e4afcdb
Fix the rest of the tests
emailtovamos 0f4ff88
Remove unused comments
emailtovamos e106889
check for code and proxy relayer + test
emailtovamos d2f9643
edge case where manager gets removed before accept
emailtovamos 979cd80
name change to be consistent
emailtovamos e094a54
delete provisional relayer when relayer is deleted
emailtovamos 1ecf7d0
Check for correct manager
emailtovamos f79183d
update comment
emailtovamos eb06f38
dlt managerToProvisionalRelayer when dltng relayer
emailtovamos e326cd0
address pr comments
emailtovamos 9fcc64b
Merge pull request #240 from bnb-chain/relayer-admin-2steprelayer
emailtovamos 1099e1b
update template
emailtovamos f012461
remove check for contracts on relayer manager
MatusKysel f21b86f
Merge branch 'develop' into relayer-admin
emailtovamos ac8698d
remove unused onlyWhitelabelRelayer
emailtovamos 0acc058
relayer: update template
MatusKysel df9bd2c
extract helper function addNewManager in tests
1ce7f9f
Merge pull request #247 from bnb-chain/relayer-admin-relayerhub-test-…
Mister-EA b13d055
relayer: fix generation scripts
MatusKysel 982ab60
Merge branch 'develop' into relayer-admin
MatusKysel File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains 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
This file contains 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
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a reason this is commented out?