Skip to content

feat: gov mod for ibc and permisionless optional relay - #271

Merged
dzmitry-lahoda merged 3 commits into
ComposableFi:masterfrom
dzmitry-lahoda-forks:dz/5
Dec 1, 2023
Merged

feat: gov mod for ibc and permisionless optional relay#271
dzmitry-lahoda merged 3 commits into
ComposableFi:masterfrom
dzmitry-lahoda-forks:dz/5

Conversation

@dzmitry-lahoda

Copy link
Copy Markdown
Contributor
  1. made gov to own what previosly vlad key did, first it ensures that security level of this chain same as piccasso, to prevent mint anything by anybody by wasm light client. second it allows to run devnet (previosly only vlad could)
  2. making other than vlad key to relay. so if there is relay filter - sure it is used, but when removed all filters - anybody can relay. on mainnet and devnet
  3. also used centauri folder for storage like any other chains do more or less.

@dzmitry-lahoda

Copy link
Copy Markdown
Contributor Author

@mina86 just FYI if need to run IBC with Composable Cosmos chain - need this PR.

Comment thread app/app.go Outdated
Comment thread x/transfermiddleware/keeper/keeper.go
@dzmitry-lahoda

Copy link
Copy Markdown
Contributor Author

running tests to see if its oke

@dzmitry-lahoda
dzmitry-lahoda requested review from RustNinja and removed request for vuong177 November 30, 2023 16:58
@dzmitry-lahoda dzmitry-lahoda changed the title feat: gov for ibc and permisionless optional relay feat: gov mod for ibc and permisionless optional relay Nov 30, 2023
@dzmitry-lahoda

Copy link
Copy Markdown
Contributor Author

@RustNinja would be awesome to have in next release.

@dzmitry-lahoda

Copy link
Copy Markdown
Contributor Author

Tested gov wasm08 uploades and permissionless relayers on devnet.

@dzmitry-lahoda
dzmitry-lahoda merged commit 387c96b into ComposableFi:master Dec 1, 2023
@blasrodri

Copy link
Copy Markdown

Thanks @dzmitry-lahoda

dzmitry-lahoda added a commit that referenced this pull request Dec 7, 2023
feat: gov mod for ibc and permisionless optional relay
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants