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.

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

Blocked list instead of allowed list for ICA host message types? #1400

Closed
3 tasks
seantking opened this issue May 20, 2022 · 2 comments
Closed
3 tasks

Blocked list instead of allowed list for ICA host message types? #1400

seantking opened this issue May 20, 2022 · 2 comments
Labels
27-interchain-accounts needs discussion Issues that need discussion before they can be worked on

Comments

@seantking
Copy link
Contributor

Summary

It appears to be a UX pain point that developers integrating the interchain account host module are required to create an allow list of all messages types that accounts on the host chain can execute.

The primary reason for this approach was to force teams using the host module to consider the implications of an ICA executing a particular message and ensure that only required messages are allowed.

Should we consider switching from an allow list to a blocked list? This would mean all messages are enabled by default.


For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate contributors tagged/assigned
@seantking seantking added needs discussion Issues that need discussion before they can be worked on 27-interchain-accounts labels May 20, 2022
@nooomski
Copy link

I'm in favor of this idea. Any messages that can be executed via ICA can also be executed via a regular account, so I don't consider this feature to add any additional risks to a chain, and would be in favor of a blocked list.

cc @mmulji-ic @yaruwangway

@crodriguezvega
Copy link
Contributor

After user feedback it has been decided to keep the whitelist, but implement the possibility to allow all messages. So closing this issue now in favor of #1479.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
27-interchain-accounts needs discussion Issues that need discussion before they can be worked on
Projects
None yet
Development

No branches or pull requests

3 participants